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

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

Issue 23537029: Save password functionality added to the save password bubble (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review 5 Created 7 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/ui/content_settings/content_setting_bubble_model.cc
diff --git a/chrome/browser/ui/content_settings/content_setting_bubble_model.cc b/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
index e97cbfd333bd921b47d5c5f512d76d67ad1c4249..6d446cca9d0eed6cac95ca43c3de5bc2643808e1 100644
--- a/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
+++ b/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
@@ -588,6 +588,8 @@ SavePasswordBubbleModel::SavePasswordBubbleModel(Delegate* delegate,
SetTitle();
}
+SavePasswordBubbleModel::~SavePasswordBubbleModel() {}
+
void SavePasswordBubbleModel::SetTitle() {
int title_id = 0;
// If the save password icon was accessed, the icon is displayed and the
@@ -601,13 +603,13 @@ void SavePasswordBubbleModel::SetTitle() {
void SavePasswordBubbleModel::OnCancelClicked() {
TabSpecificContentSettings* content_settings =
TabSpecificContentSettings::FromWebContents(web_contents());
- content_settings->PasswordFormBlacklisted();
+ content_settings->set_password_action(PasswordFormManager::BLACKLIST);
}
void SavePasswordBubbleModel::OnSaveClicked() {
TabSpecificContentSettings* content_settings =
TabSpecificContentSettings::FromWebContents(web_contents());
- content_settings->PasswordAccepted();
+ content_settings->set_password_action(PasswordFormManager::SAVE);
}
// The model of the content settings bubble for media settings.
@@ -621,7 +623,6 @@ class ContentSettingMediaStreamBubbleModel
virtual ~ContentSettingMediaStreamBubbleModel();
private:
- // Sets the title of the bubble.
void SetTitle();
// Sets the data for the radio buttons of the bubble.
void SetRadioGroup();

Powered by Google App Engine
This is Rietveld 408576698