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

Unified Diff: chrome/browser/ui/content_settings/content_setting_bubble_model.h

Issue 10821059: Add done-clicked notification to content settings bubble model. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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/content_settings/content_setting_bubble_model.h
diff --git a/chrome/browser/ui/content_settings/content_setting_bubble_model.h b/chrome/browser/ui/content_settings/content_setting_bubble_model.h
index 9e4377b0cf8d289978940554e89cae1bbcaf1aae..e661cdfa366d0347f2a0e9ce038ddeb3eb008871 100644
--- a/chrome/browser/ui/content_settings/content_setting_bubble_model.h
+++ b/chrome/browser/ui/content_settings/content_setting_bubble_model.h
@@ -94,6 +94,10 @@ class ContentSettingBubbleModel : public content::NotificationObserver {
virtual void OnCustomLinkClicked() {}
virtual void OnManageLinkClicked() {}
+ // Called by the view code when the bubble is closed by the user using the
+ // Done button.
+ virtual void OnDoneClicked() {}
+
protected:
ContentSettingBubbleModel(TabContents* tab_contents, Profile* profile,
ContentSettingsType content_type);
@@ -163,6 +167,7 @@ class ContentSettingRPHBubbleModel : public ContentSettingTitleAndLinkModel {
ContentSettingsType content_type);
virtual void OnRadioClicked(int radio_index) OVERRIDE;
+ virtual void OnDoneClicked() OVERRIDE;
private:
// These states must match the order of appearance of the radio buttons

Powered by Google App Engine
This is Rietveld 408576698