ThinBridge / Chronos

Chromiumベースの業務ブラウザ
Other
5 stars 3 forks source link

Suppress warning C23812 of MediaAccessPermission #174

Closed HashidaTKS closed 3 months ago

HashidaTKS commented 3 months ago

Which issue(s) this PR fixes:

42

What this PR does / why we need it:

MediaAccessPermissionの以下の警告を解消。

C:\gitdir\Chronos\sbcommon.h(907): warning C26812: 列挙型 'AppSettings::MediaAccessPermission' は対象範囲外です。'enum' (Enum.3) より 'enum class' を優先します。

元々列挙型としての相応しい使い方がされていそうなので、enumからenum classを使うように修正。

How to verify the fixed issue:

The steps to verify:

警告の解消の確認

  1. Code Analysisを実行する。また、Chronosをリビルドする。

リグレッションテスト

ChronosDefault.confにて、EnableMediaAccessPermissionとEnableMediaAccessPermissionを設定してメディアアクセスの動作を確認する。

Expected result:

警告の解消の確認

C:\gitdir\Chronos\sbcommon.h(907): warning C26812: 列挙型 'AppSettings::MediaAccessPermission' は対象範囲外です。'enum' (Enum.3) より 'enum class' を優先します。

上記の警告が出ないこと。

リグレッションテスト

メディアアクセスの動作が、テスト手順に記載の期待値の通りであること。