Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(580)

Unified Diff: chrome/browser/plugins/flash_permission_context.cc

Issue 2378573005: [HBD] Blanket BLOCK on all non-HTTP(s) and non-FILE URLs for Flash. (Closed)
Patch Set: fix formatting Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/plugins/flash_permission_context.cc
diff --git a/chrome/browser/plugins/flash_permission_context.cc b/chrome/browser/plugins/flash_permission_context.cc
index 941edbb2d2b36710139fbb330bb3b6594ad963c0..7efcd0e780a414dd172c55aa20ec541000d83d05 100644
--- a/chrome/browser/plugins/flash_permission_context.cc
+++ b/chrome/browser/plugins/flash_permission_context.cc
@@ -12,6 +12,7 @@
#include "chrome/browser/ui/website_settings/website_settings_infobar_delegate.h"
#include "components/content_settings/core/browser/host_content_settings_map.h"
#include "content/public/browser/render_frame_host.h"
+#include "url/origin.h"
FlashPermissionContext::FlashPermissionContext(Profile* profile)
: PermissionContextBase(profile,
@@ -24,8 +25,8 @@ ContentSetting FlashPermissionContext::GetPermissionStatus(
const GURL& requesting_origin,
const GURL& embedding_origin) const {
ContentSetting flash_setting = PluginUtils::GetFlashPluginContentSetting(
- HostContentSettingsMapFactory::GetForProfile(profile()), embedding_origin,
- requesting_origin, nullptr);
+ HostContentSettingsMapFactory::GetForProfile(profile()),
+ url::Origin(embedding_origin), requesting_origin, nullptr);
flash_setting = PluginsFieldTrial::EffectiveContentSetting(
CONTENT_SETTINGS_TYPE_PLUGINS, flash_setting);
if (flash_setting == CONTENT_SETTING_DETECT_IMPORTANT_CONTENT)

Powered by Google App Engine
This is Rietveld 408576698