Implement Desktop UI for Site Muting
This CL adds desktop UI that uses the sound content setting to mute and
unmute WebContents. This is hidden behind a feature flag
(kSoundContentSetting).
Bug: 743001
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: If685718e2b5f69d8480b40d5d80a8a64cf7ca13e
Reviewed-on: https://chromium-review.googlesource.com/580112
Reviewed-by: Tommy Li <[email protected]>
Reviewed-by: Scott Violet <[email protected]>
Reviewed-by: Yuri Wiitala <[email protected]>
Reviewed-by: Mounir Lamouri <[email protected]>
Commit-Queue: Tommy Steimel <[email protected]>
Cr-Commit-Position: refs/heads/master@{#496881}diff --git a/chrome/browser/ui/page_info/page_info.cc b/chrome/browser/ui/page_info/page_info.cc
index 6c9f3c7..a4a4392 100644
--- a/chrome/browser/ui/page_info/page_info.cc
+++ b/chrome/browser/ui/page_info/page_info.cc
@@ -114,6 +114,7 @@
CONTENT_SETTINGS_TYPE_POPUPS,
CONTENT_SETTINGS_TYPE_ADS,
CONTENT_SETTINGS_TYPE_BACKGROUND_SYNC,
+ CONTENT_SETTINGS_TYPE_SOUND,
CONTENT_SETTINGS_TYPE_AUTOMATIC_DOWNLOADS,
CONTENT_SETTINGS_TYPE_AUTOPLAY,
CONTENT_SETTINGS_TYPE_MIDI_SYSEX,
@@ -143,6 +144,9 @@
nullptr) != nullptr;
}
+ if (type == CONTENT_SETTINGS_TYPE_SOUND)
+ return base::FeatureList::IsEnabled(features::kSoundContentSetting);
+
return true;
}
