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

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

Issue 2945243002: Permissions: Allow PermissionManager to return more PermissionStatusSources.
Patch Set: Cleanup. Created 3 years, 6 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/chrome_plugin_service_filter.cc
diff --git a/chrome/browser/plugins/chrome_plugin_service_filter.cc b/chrome/browser/plugins/chrome_plugin_service_filter.cc
index 1783a101aa024a5aa39a7bd7c335427398d47093..4611bbd7025c7be12601ebbb3eac91b2b7e1577c 100644
--- a/chrome/browser/plugins/chrome_plugin_service_filter.cc
+++ b/chrome/browser/plugins/chrome_plugin_service_filter.cc
@@ -13,6 +13,7 @@
#include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/content_settings/host_content_settings_map_factory.h"
#include "chrome/browser/engagement/site_engagement_service.h"
+#include "chrome/browser/permissions/permission_result.h"
#include "chrome/browser/plugins/flash_temporary_permission_tracker.h"
#include "chrome/browser/plugins/plugin_finder.h"
#include "chrome/browser/plugins/plugin_metadata.h"
@@ -223,8 +224,10 @@ bool ChromePluginServiceFilter::IsPluginAvailable(
bool is_managed = false;
HostContentSettingsMap* settings_map =
context_info_it->second->host_content_settings_map.get();
- ContentSetting flash_setting = PluginUtils::GetFlashPluginContentSetting(
- settings_map, main_frame_origin, plugin_content_url, &is_managed);
+ ContentSetting flash_setting =
+ PluginUtils::GetFlashPluginContentSetting(
+ settings_map, main_frame_origin, plugin_content_url, &is_managed)
+ .content_setting;
flash_setting = PluginsFieldTrial::EffectiveContentSetting(
settings_map, CONTENT_SETTINGS_TYPE_PLUGINS, flash_setting);
double engagement = SiteEngagementService::GetScoreFromSettings(
« no previous file with comments | « chrome/browser/permissions/permission_util.cc ('k') | chrome/browser/plugins/flash_download_interception.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698