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

Unified Diff: chrome/browser/ui/website_settings/permission_menu_model.h

Issue 2408613002: Replace kPreferHtmlOverPlugins feature checks with PluginUtils::ShouldPreferHtmlOverPlugi… (Closed)
Patch Set: Replace remaining kPreferHtmlOverPlugins feature checks with PluginUtils::ShouldPreferHtmlOverPlugi… Created 4 years, 2 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/ui/website_settings/permission_menu_model.h
diff --git a/chrome/browser/ui/website_settings/permission_menu_model.h b/chrome/browser/ui/website_settings/permission_menu_model.h
index 6ab56963d6e797ee23d3526d29e97150afa967fd..86cab8b8bd292e2f1de87c673bca31527aeaacc6 100644
--- a/chrome/browser/ui/website_settings/permission_menu_model.h
+++ b/chrome/browser/ui/website_settings/permission_menu_model.h
@@ -12,6 +12,9 @@
#include "ui/base/models/simple_menu_model.h"
#include "url/gurl.h"
+class HostContentSettingsMap;
+class Profile;
+
class PermissionMenuModel : public ui::SimpleMenuModel,
public ui::SimpleMenuModel::Delegate {
public:
@@ -19,14 +22,16 @@ class PermissionMenuModel : public ui::SimpleMenuModel,
ChangeCallback;
// Create a new menu model for permission settings.
- PermissionMenuModel(const GURL& url,
+ PermissionMenuModel(Profile* profile,
+ const GURL& url,
const WebsiteSettingsUI::PermissionInfo& info,
const ChangeCallback& callback);
// Creates a special-case menu model that only has the allow and block
// options. It does not track a permission type. |setting| is the
// initial selected option. It must be either CONTENT_SETTING_ALLOW or
// CONTENT_SETTING_BLOCK.
- PermissionMenuModel(const GURL& url,
+ PermissionMenuModel(Profile* profile,
+ const GURL& url,
ContentSetting setting,
const ChangeCallback& callback);
~PermissionMenuModel() override;
@@ -37,6 +42,8 @@ class PermissionMenuModel : public ui::SimpleMenuModel,
void ExecuteCommand(int command_id, int event_flags) override;
private:
+ HostContentSettingsMap* host_content_settings_map_;
+
// The permission info represented by the menu model.
WebsiteSettingsUI::PermissionInfo permission_;

Powered by Google App Engine
This is Rietveld 408576698