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

Side by Side Diff: chrome/browser/password_manager/save_password_infobar_delegate.cc

Issue 967193002: Credential saving clank infobar for Smart Lock. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@reusable_more_button
Patch Set: Created 5 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/password_manager/save_password_infobar_delegate.h" 5 #include "chrome/browser/password_manager/save_password_infobar_delegate.h"
6 6
7 #include "base/metrics/histogram.h" 7 #include "base/metrics/histogram.h"
8 #include "base/strings/utf_string_conversions.h"
9 #include "chrome/browser/infobars/infobar_service.h" 8 #include "chrome/browser/infobars/infobar_service.h"
10 #include "chrome/grit/chromium_strings.h" 9 #include "chrome/grit/chromium_strings.h"
11 #include "chrome/grit/generated_resources.h" 10 #include "chrome/grit/generated_resources.h"
12 #include "components/infobars/core/infobar.h" 11 #include "components/infobars/core/infobar.h"
13 #include "components/password_manager/core/browser/password_form_manager.h" 12 #include "components/password_manager/core/browser/password_manager_client.h"
14 #include "content/public/browser/navigation_entry.h"
15 #include "content/public/browser/web_contents.h" 13 #include "content/public/browser/web_contents.h"
16 #include "grit/theme_resources.h" 14 #include "grit/theme_resources.h"
17 #include "ui/base/l10n/l10n_util.h" 15 #include "ui/base/l10n/l10n_util.h"
18 16
17 #if defined(OS_ANDROID)
18 #include "chrome/browser/ui/android/infobars/save_password_infobar.h"
19 #endif
20
21 namespace {
22
23 int GetCancelButtonText(password_manager::CredentialSourceType source_type) {
24 return source_type ==
25 password_manager::CredentialSourceType::CREDENTIAL_SOURCE_API
26 ? IDS_PASSWORD_MANAGER_SAVE_PASSWORD_SMART_LOCK_NO_THANKS_BUTTON
27 : IDS_PASSWORD_MANAGER_BLACKLIST_BUTTON;
28 }
29
30 } // namespace
31
19 // static 32 // static
20 void SavePasswordInfoBarDelegate::Create( 33 void SavePasswordInfoBarDelegate::Create(
21 content::WebContents* web_contents, 34 content::WebContents* web_contents,
22 scoped_ptr<password_manager::PasswordFormManager> form_to_save, 35 scoped_ptr<password_manager::PasswordFormManager> form_to_save,
23 const std::string& uma_histogram_suffix) { 36 const std::string& uma_histogram_suffix,
37 password_manager::CredentialSourceType source_type) {
24 InfoBarService* infobar_service = 38 InfoBarService* infobar_service =
25 InfoBarService::FromWebContents(web_contents); 39 InfoBarService::FromWebContents(web_contents);
26 infobar_service->AddInfoBar(infobar_service->CreateConfirmInfoBar( 40 SavePasswordInfoBarDelegate* infobar_delegate =
27 scoped_ptr<ConfirmInfoBarDelegate>(new SavePasswordInfoBarDelegate( 41 new SavePasswordInfoBarDelegate(
28 form_to_save.Pass(), uma_histogram_suffix)))); 42 form_to_save.Pass(), uma_histogram_suffix, source_type);
43 #if defined(OS_ANDROID)
44 // For Android in case of smart lock we need different appearance of infobar.
45 scoped_ptr<infobars::InfoBar> infobar =
46 make_scoped_ptr(new SavePasswordInfoBar(
47 scoped_ptr<SavePasswordInfoBarDelegate>(infobar_delegate)));
48 #else
49 // For desktop we'll keep using the ConfirmInfobar.
50 scoped_ptr<infobars::InfoBar> infobar(infobar_service->CreateConfirmInfoBar(
51 scoped_ptr<ConfirmInfoBarDelegate>(infobar_delegate)));
52 #endif
53 infobar_service->AddInfoBar(infobar.Pass());
29 } 54 }
30 55
31 SavePasswordInfoBarDelegate::~SavePasswordInfoBarDelegate() { 56 SavePasswordInfoBarDelegate::~SavePasswordInfoBarDelegate() {
32 UMA_HISTOGRAM_ENUMERATION("PasswordManager.InfoBarResponse", 57 UMA_HISTOGRAM_ENUMERATION("PasswordManager.InfoBarResponse",
33 infobar_response_, 58 infobar_response_,
34 password_manager::metrics_util::NUM_RESPONSE_TYPES); 59 password_manager::metrics_util::NUM_RESPONSE_TYPES);
35 60
36 password_manager::metrics_util::LogUIDismissalReason(infobar_response_); 61 password_manager::metrics_util::LogUIDismissalReason(infobar_response_);
37 62
38 // The shortest period for which the prompt needs to live, so that we don't 63 // The shortest period for which the prompt needs to live, so that we don't
39 // consider it killed prematurely, as might happen, e.g., if a pre-rendered 64 // consider it killed prematurely, as might happen, e.g., if a pre-rendered
40 // page gets swapped in (and the current WebContents is destroyed). 65 // page gets swapped in (and the current WebContents is destroyed).
41 const base::TimeDelta kMinimumPromptDisplayTime = 66 const base::TimeDelta kMinimumPromptDisplayTime =
42 base::TimeDelta::FromSeconds(1); 67 base::TimeDelta::FromSeconds(1);
43 68
44 if (!uma_histogram_suffix_.empty()) { 69 if (!uma_histogram_suffix_.empty()) {
45 password_manager::metrics_util::LogUMAHistogramEnumeration( 70 password_manager::metrics_util::LogUMAHistogramEnumeration(
46 "PasswordManager.SavePasswordPromptResponse_" + uma_histogram_suffix_, 71 "PasswordManager.SavePasswordPromptResponse_" + uma_histogram_suffix_,
47 infobar_response_, 72 infobar_response_,
48 password_manager::metrics_util::NUM_RESPONSE_TYPES); 73 password_manager::metrics_util::NUM_RESPONSE_TYPES);
49 password_manager::metrics_util::LogUMAHistogramBoolean( 74 password_manager::metrics_util::LogUMAHistogramBoolean(
50 "PasswordManager.SavePasswordPromptDisappearedQuickly_" + 75 "PasswordManager.SavePasswordPromptDisappearedQuickly_" +
51 uma_histogram_suffix_, 76 uma_histogram_suffix_,
52 timer_.Elapsed() < kMinimumPromptDisplayTime); 77 timer_.Elapsed() < kMinimumPromptDisplayTime);
53 } 78 }
54 } 79 }
55 80
56 SavePasswordInfoBarDelegate::SavePasswordInfoBarDelegate( 81 SavePasswordInfoBarDelegate::SavePasswordInfoBarDelegate(
57 scoped_ptr<password_manager::PasswordFormManager> form_to_save, 82 scoped_ptr<password_manager::PasswordFormManager> form_to_save,
58 const std::string& uma_histogram_suffix) 83 const std::string& uma_histogram_suffix,
84 password_manager::CredentialSourceType source_type)
59 : ConfirmInfoBarDelegate(), 85 : ConfirmInfoBarDelegate(),
60 form_to_save_(form_to_save.Pass()), 86 form_to_save_(form_to_save.Pass()),
61 infobar_response_(password_manager::metrics_util::NO_RESPONSE), 87 infobar_response_(password_manager::metrics_util::NO_RESPONSE),
62 uma_histogram_suffix_(uma_histogram_suffix) { 88 uma_histogram_suffix_(uma_histogram_suffix),
89 source_type_(source_type) {
63 if (!uma_histogram_suffix_.empty()) { 90 if (!uma_histogram_suffix_.empty()) {
64 password_manager::metrics_util::LogUMAHistogramBoolean( 91 password_manager::metrics_util::LogUMAHistogramBoolean(
65 "PasswordManager.SavePasswordPromptDisplayed_" + uma_histogram_suffix_, 92 "PasswordManager.SavePasswordPromptDisplayed_" + uma_histogram_suffix_,
66 true); 93 true);
67 } 94 }
68 } 95 }
69 96
97 bool SavePasswordInfoBarDelegate::ShouldShowMoreButton() {
98 return source_type_ ==
99 password_manager::CredentialSourceType::CREDENTIAL_SOURCE_API;
100 }
101
70 infobars::InfoBarDelegate::Type 102 infobars::InfoBarDelegate::Type
71 SavePasswordInfoBarDelegate::GetInfoBarType() const { 103 SavePasswordInfoBarDelegate::GetInfoBarType() const {
72 return PAGE_ACTION_TYPE; 104 return PAGE_ACTION_TYPE;
73 } 105 }
74 106
75 infobars::InfoBarDelegate::InfoBarAutomationType 107 infobars::InfoBarDelegate::InfoBarAutomationType
76 SavePasswordInfoBarDelegate::GetInfoBarAutomationType() const { 108 SavePasswordInfoBarDelegate::GetInfoBarAutomationType() const {
77 return PASSWORD_INFOBAR; 109 return PASSWORD_INFOBAR;
78 } 110 }
79 111
80 int SavePasswordInfoBarDelegate::GetIconID() const { 112 int SavePasswordInfoBarDelegate::GetIconID() const {
81 return IDR_INFOBAR_SAVE_PASSWORD; 113 return IDR_INFOBAR_SAVE_PASSWORD;
82 } 114 }
83 115
84 bool SavePasswordInfoBarDelegate::ShouldExpire( 116 bool SavePasswordInfoBarDelegate::ShouldExpire(
85 const NavigationDetails& details) const { 117 const NavigationDetails& details) const {
86 return !details.is_redirect && 118 return !details.is_redirect &&
87 infobars::InfoBarDelegate::ShouldExpire(details); 119 infobars::InfoBarDelegate::ShouldExpire(details);
88 } 120 }
89 121
90 void SavePasswordInfoBarDelegate::InfoBarDismissed() { 122 void SavePasswordInfoBarDelegate::InfoBarDismissed() {
91 DCHECK(form_to_save_.get()); 123 DCHECK(form_to_save_.get());
92 infobar_response_ = password_manager::metrics_util::INFOBAR_DISMISSED; 124 infobar_response_ = password_manager::metrics_util::INFOBAR_DISMISSED;
93 } 125 }
94 126
95 base::string16 SavePasswordInfoBarDelegate::GetMessageText() const { 127 base::string16 SavePasswordInfoBarDelegate::GetMessageText() const {
96 return l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_SAVE_PASSWORD_PROMPT); 128 return l10n_util::GetStringUTF16(
129 (source_type_ ==
130 password_manager::CredentialSourceType::CREDENTIAL_SOURCE_API)
131 ? IDS_PASSWORD_MANAGER_SAVE_PASSWORD_SMART_LOCK_PROMPT
132 : IDS_PASSWORD_MANAGER_SAVE_PASSWORD_PROMPT);
97 } 133 }
98 134
99 base::string16 SavePasswordInfoBarDelegate::GetButtonLabel( 135 base::string16 SavePasswordInfoBarDelegate::GetButtonLabel(
100 InfoBarButton button) const { 136 InfoBarButton button) const {
101 return l10n_util::GetStringUTF16((button == BUTTON_OK) ? 137 return l10n_util::GetStringUTF16((button == BUTTON_OK)
102 IDS_PASSWORD_MANAGER_SAVE_BUTTON : IDS_PASSWORD_MANAGER_BLACKLIST_BUTTON); 138 ? IDS_PASSWORD_MANAGER_SAVE_BUTTON
139 : GetCancelButtonText(source_type_));
103 } 140 }
104 141
105 bool SavePasswordInfoBarDelegate::Accept() { 142 bool SavePasswordInfoBarDelegate::Accept() {
106 DCHECK(form_to_save_.get()); 143 DCHECK(form_to_save_.get());
107 form_to_save_->Save(); 144 form_to_save_->Save();
108 infobar_response_ = password_manager::metrics_util::REMEMBER_PASSWORD; 145 infobar_response_ = password_manager::metrics_util::REMEMBER_PASSWORD;
109 return true; 146 return true;
110 } 147 }
111 148
112 bool SavePasswordInfoBarDelegate::Cancel() { 149 bool SavePasswordInfoBarDelegate::Cancel() {
113 DCHECK(form_to_save_.get()); 150 DCHECK(form_to_save_.get());
114 form_to_save_->PermanentlyBlacklist(); 151 if (source_type_ ==
115 infobar_response_ = password_manager::metrics_util::NEVER_REMEMBER_PASSWORD; 152 password_manager::CredentialSourceType::CREDENTIAL_SOURCE_API) {
153 form_to_save_->PermanentlyBlacklist();
154 infobar_response_ = password_manager::metrics_util::NEVER_REMEMBER_PASSWORD;
155 } else {
156 InfoBarDismissed();
157 }
116 return true; 158 return true;
117 } 159 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698