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

Side by Side Diff: components/autofill/content/browser/autofill_cc_infobar_delegate.cc

Issue 710453002: [Autofill] Componentize AutofillCCInfoBarDelegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Abstracting the call to WebContents::OpenURL() throuh the AutofillDriver. Created 6 years, 1 month 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/autofill/autofill_cc_infobar_delegate.h" 5 #include "components/autofill/content/browser/autofill_cc_infobar_delegate.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "chrome/browser/infobars/infobar_service.h" 8 #include "components/autofill/core/browser/autofill_driver.h"
9 #include "chrome/grit/generated_resources.h"
10 #include "chrome/grit/google_chrome_strings.h"
11 #include "components/autofill/core/browser/credit_card.h" 9 #include "components/autofill/core/browser/credit_card.h"
12 #include "components/autofill/core/browser/personal_data_manager.h" 10 #include "components/autofill/core/browser/personal_data_manager.h"
13 #include "components/autofill/core/common/autofill_constants.h" 11 #include "components/autofill/core/common/autofill_constants.h"
14 #include "components/infobars/core/infobar.h" 12 #include "components/infobars/core/infobar.h"
15 #include "content/public/browser/page_navigator.h" 13 #include "components/infobars/core/infobar_manager.h"
16 #include "content/public/browser/web_contents.h" 14 #include "grit/components_scaled_resources.h"
17 #include "content/public/browser/web_contents_delegate.h"
18 #include "grit/components_strings.h" 15 #include "grit/components_strings.h"
19 #include "grit/theme_resources.h"
20 #include "ui/base/l10n/l10n_util.h" 16 #include "ui/base/l10n/l10n_util.h"
21 17
22 namespace autofill { 18 namespace autofill {
23 19
24 // static 20 // static
25 void AutofillCCInfoBarDelegate::Create( 21 void AutofillCCInfoBarDelegate::Create(
26 InfoBarService* infobar_service, 22 infobars::InfoBarManager* infobar_manager,
23 const AutofillDriver* autofill_driver,
27 const AutofillMetrics* metric_logger, 24 const AutofillMetrics* metric_logger,
28 const base::Closure& save_card_callback) { 25 const base::Closure& save_card_callback) {
29 infobar_service->AddInfoBar(ConfirmInfoBarDelegate::CreateInfoBar( 26 infobar_manager->AddInfoBar(ConfirmInfoBarDelegate::CreateInfoBar(
30 scoped_ptr<ConfirmInfoBarDelegate>(new AutofillCCInfoBarDelegate( 27 scoped_ptr<ConfirmInfoBarDelegate>(new AutofillCCInfoBarDelegate(
31 metric_logger, save_card_callback)))); 28 autofill_driver, metric_logger, save_card_callback))));
32 } 29 }
33 30
34 AutofillCCInfoBarDelegate::AutofillCCInfoBarDelegate( 31 AutofillCCInfoBarDelegate::AutofillCCInfoBarDelegate(
32 const AutofillDriver* autofill_driver,
35 const AutofillMetrics* metric_logger, 33 const AutofillMetrics* metric_logger,
36 const base::Closure& save_card_callback) 34 const base::Closure& save_card_callback)
37 : ConfirmInfoBarDelegate(), 35 : ConfirmInfoBarDelegate(),
36 autofill_driver_(autofill_driver),
38 metric_logger_(metric_logger), 37 metric_logger_(metric_logger),
39 save_card_callback_(save_card_callback), 38 save_card_callback_(save_card_callback),
40 had_user_interaction_(false) { 39 had_user_interaction_(false) {
41 metric_logger->LogCreditCardInfoBarMetric(AutofillMetrics::INFOBAR_SHOWN); 40 metric_logger->LogCreditCardInfoBarMetric(AutofillMetrics::INFOBAR_SHOWN);
42 } 41 }
43 42
44 AutofillCCInfoBarDelegate::~AutofillCCInfoBarDelegate() { 43 AutofillCCInfoBarDelegate::~AutofillCCInfoBarDelegate() {
45 if (!had_user_interaction_) 44 if (!had_user_interaction_)
46 LogUserAction(AutofillMetrics::INFOBAR_IGNORED); 45 LogUserAction(AutofillMetrics::INFOBAR_IGNORED);
47 } 46 }
(...skipping 26 matching lines...) Expand all
74 // get a chance to answer the question. 73 // get a chance to answer the question.
75 return false; 74 return false;
76 } 75 }
77 76
78 base::string16 AutofillCCInfoBarDelegate::GetMessageText() const { 77 base::string16 AutofillCCInfoBarDelegate::GetMessageText() const {
79 return l10n_util::GetStringUTF16(IDS_AUTOFILL_CC_INFOBAR_TEXT); 78 return l10n_util::GetStringUTF16(IDS_AUTOFILL_CC_INFOBAR_TEXT);
80 } 79 }
81 80
82 base::string16 AutofillCCInfoBarDelegate::GetButtonLabel( 81 base::string16 AutofillCCInfoBarDelegate::GetButtonLabel(
83 InfoBarButton button) const { 82 InfoBarButton button) const {
84 return l10n_util::GetStringUTF16((button == BUTTON_OK) ? 83 return l10n_util::GetStringUTF16((button == BUTTON_OK)
85 IDS_AUTOFILL_CC_INFOBAR_ACCEPT : IDS_AUTOFILL_CC_INFOBAR_DENY); 84 ? IDS_AUTOFILL_CC_INFOBAR_ACCEPT
85 : IDS_AUTOFILL_CC_INFOBAR_DENY);
Peter Kasting 2014/11/21 19:42:17 Nit: Please leave this formatting as it was before
Pritam Nikam 2014/11/24 14:45:53 Done. "git cl format" would have disturbed the for
86 } 86 }
87 87
88 bool AutofillCCInfoBarDelegate::Accept() { 88 bool AutofillCCInfoBarDelegate::Accept() {
89 save_card_callback_.Run(); 89 save_card_callback_.Run();
90 save_card_callback_.Reset(); 90 save_card_callback_.Reset();
91 LogUserAction(AutofillMetrics::INFOBAR_ACCEPTED); 91 LogUserAction(AutofillMetrics::INFOBAR_ACCEPTED);
92 return true; 92 return true;
93 } 93 }
94 94
95 bool AutofillCCInfoBarDelegate::Cancel() { 95 bool AutofillCCInfoBarDelegate::Cancel() {
96 LogUserAction(AutofillMetrics::INFOBAR_DENIED); 96 LogUserAction(AutofillMetrics::INFOBAR_DENIED);
97 return true; 97 return true;
98 } 98 }
99 99
100 base::string16 AutofillCCInfoBarDelegate::GetLinkText() const { 100 base::string16 AutofillCCInfoBarDelegate::GetLinkText() const {
101 return l10n_util::GetStringUTF16(IDS_LEARN_MORE); 101 return l10n_util::GetStringUTF16(IDS_LEARN_MORE);
102 } 102 }
103 103
104 bool AutofillCCInfoBarDelegate::LinkClicked(WindowOpenDisposition disposition) { 104 bool AutofillCCInfoBarDelegate::LinkClicked(WindowOpenDisposition disposition) {
105 InfoBarService::WebContentsFromInfoBar(infobar())->OpenURL( 105 autofill_driver_->LinkClicked(disposition);
106 content::OpenURLParams(
107 GURL(autofill::kHelpURL), content::Referrer(),
108 (disposition == CURRENT_TAB) ? NEW_FOREGROUND_TAB : disposition,
109 ui::PAGE_TRANSITION_LINK, false));
110 return false; 106 return false;
111 } 107 }
112 108
113 } // namespace autofill 109 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698