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

Unified Diff: chrome/browser/ui/webui/options/website_settings_handler.h

Issue 380893005: Add an option page for searching and managing resources and permissions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 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/webui/options/website_settings_handler.h
diff --git a/chrome/browser/ui/webui/options/website_settings_handler.h b/chrome/browser/ui/webui/options/website_settings_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..d944beadd5695d4d96ec366bd48131bc33ab9745
--- /dev/null
+++ b/chrome/browser/ui/webui/options/website_settings_handler.h
@@ -0,0 +1,46 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_WEBUI_OPTIONS_WEBSITE_SETTINGS_HANDLER_H_
+#define CHROME_BROWSER_UI_WEBUI_OPTIONS_WEBSITE_SETTINGS_HANDLER_H_
+
Dan Beam 2014/07/10 02:57:25 #include <string> #include "base/macros.h" #inclu
Daniel Nishi 2014/07/10 19:00:10 Done.
+#include "chrome/browser/content_settings/host_content_settings_map.h"
+#include "chrome/browser/ui/webui/options/options_ui.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+
+namespace options {
+
+class WebsiteSettingsHandler : public OptionsPageUIHandler,
+ public content::NotificationObserver {
+ public:
+ WebsiteSettingsHandler();
+ virtual ~WebsiteSettingsHandler();
+
+ // OptionsPageUIHandler implementation.
+ virtual void GetLocalizedValues(
+ base::DictionaryValue* localized_strings) OVERRIDE;
+ virtual void RegisterMessages() OVERRIDE;
+
+ // content::NotificationObserver implementation.
+ virtual void Observe(int type,
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details) OVERRIDE;
+
+ private:
+ // Update the page with all origins for a given Content Setting.
Dan Beam 2014/07/10 02:57:24 list what |args| contains in each of these handler
Daniel Nishi 2014/07/10 19:00:10 Done.
+ void UpdateOrigins(const base::ListValue* args);
Dan Beam 2014/07/10 02:57:24 nit: if it's chrome.send('messageName') name the C
Daniel Nishi 2014/07/10 19:00:10 Done.
+
+ // Update the page with all origins given a filter string.
+ void UpdateSearchResults(const base::ListValue* args);
+
Dan Beam 2014/07/10 02:57:25 doc comment
Daniel Nishi 2014/07/10 19:00:10 Done.
+ void UpdateOrigins_(ContentSettingsType content_type,
Dan Beam 2014/07/10 02:57:25 no _ at the end for private methods
Daniel Nishi 2014/07/10 19:00:10 Done.
+ const std::string& filter);
+
+ ContentSettingsType last_setting_;
Dan Beam 2014/07/10 02:57:25 DISALLOW_COPY_AND_ASSIGN(WebsiteSettingsHandler);
Daniel Nishi 2014/07/10 19:00:09 Done.
+};
+
+} // namespace options
+
+#endif // CHROME_BROWSER_UI_WEBUI_OPTIONS_WEBSITE_SETTINGS_HANDLER_H_

Powered by Google App Engine
This is Rietveld 408576698