Website settings for Fullscreen should behave like Geolocation or midi sysex.
The permission handling for fullscreen has changed in order to take into
account embedder and iframe origins. The website settings UI has to take
that into account.
BUG=458462
Review URL: https://codereview.chromium.org/913993005
Cr-Commit-Position: refs/heads/master@{#316801}
diff --git a/chrome/browser/ui/website_settings/website_settings.cc b/chrome/browser/ui/website_settings/website_settings.cc
index b86be9bb..699cf12 100644
--- a/chrome/browser/ui/website_settings/website_settings.cc
+++ b/chrome/browser/ui/website_settings/website_settings.cc
@@ -264,6 +264,7 @@
switch (type) {
case CONTENT_SETTINGS_TYPE_GEOLOCATION:
case CONTENT_SETTINGS_TYPE_MIDI_SYSEX:
+ case CONTENT_SETTINGS_TYPE_FULLSCREEN:
// TODO(markusheintz): The rule we create here should also change the
// location permission for iframed content.
primary_pattern = ContentSettingsPattern::FromURLNoWildcard(site_url_);
@@ -277,7 +278,6 @@
case CONTENT_SETTINGS_TYPE_JAVASCRIPT:
case CONTENT_SETTINGS_TYPE_PLUGINS:
case CONTENT_SETTINGS_TYPE_POPUPS:
- case CONTENT_SETTINGS_TYPE_FULLSCREEN:
case CONTENT_SETTINGS_TYPE_MOUSELOCK:
case CONTENT_SETTINGS_TYPE_AUTOMATIC_DOWNLOADS:
case CONTENT_SETTINGS_TYPE_PUSH_MESSAGING:
