Interactive autofill: Adds footnote view to accept legal documents in the UI.

chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/ui/autofill/autofill_dialog_controller_impl.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "base/string_split.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autofill/autofill_country.h"
 #include "chrome/browser/autofill/autofill_manager.h"
 #include "chrome/browser/autofill/autofill_type.h"
 #include "chrome/browser/autofill/personal_data_manager.h"
 #include "chrome/browser/autofill/personal_data_manager_factory.h"
 #include "chrome/browser/autofill/validation.h"
 #include "chrome/browser/autofill/wallet/full_wallet.h"
 #include "chrome/browser/autofill/wallet/wallet_items.h"
 #include "chrome/browser/autofill/wallet/wallet_service_url.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/autofill/autofill_dialog_view.h"
+#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/browser_navigator.h"
 #include "chrome/common/form_data.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/url_constants.h"
+#include "googleurl/src/gurl.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "net/base/cert_status_flags.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
+#include "ui/base/window_open_disposition.h"
 
 namespace autofill {
 
 namespace {
 
+// TODO(dbeam): real URLs.
+const char kPrivacyPolicyUrl[] = "chrome://policy";
+const char kTermsOfServiceUrl[] = "chrome://terms";
+
 // Returns true if |input| should be shown when |field| has been requested.
 bool InputTypeMatchesFieldType(const DetailInput& input,
                                const AutofillField& field) {
   // If any credit card expiration info is asked for, show both month and year
   // inputs.
   if (field.type() == CREDIT_CARD_EXP_4_DIGIT_YEAR ||
       field.type() == CREDIT_CARD_EXP_2_DIGIT_YEAR ||
       field.type() == CREDIT_CARD_EXP_DATE_4_DIGIT_YEAR ||
       field.type() == CREDIT_CARD_EXP_DATE_2_DIGIT_YEAR ||
       field.type() == CREDIT_CARD_EXP_MONTH) {
@@ skipping 211 matching lines @@
 string16 AutofillDialogControllerImpl::SignInText() const {
   // TODO(abodenha): real strings and l10n.
   return string16(ASCIIToUTF16("Sign in to use Google Wallet"));
 }
 
 string16 AutofillDialogControllerImpl::CancelSignInText() const {
   // TODO(abodenha): real strings and l10n.
   return string16(ASCIIToUTF16("Don't sign in."));
 }
 
+string16 AutofillDialogControllerImpl::SaveLocallyText() const {
+  return l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_SAVE_LOCALLY_CHECKBOX);
+}
+
+string16 AutofillDialogControllerImpl::ProgressBarText() const {
+  return l10n_util::GetStringUTF16(
+      IDS_AUTOFILL_DIALOG_AUTOCHECKOUT_PROGRESS_BAR);
+}
+
+std::pair<std::vector<string16>, string16>
+    AutofillDialogControllerImpl::FootnoteTextParts() const {
+  string16 full_translation = l10n_util::GetStringFUTF16(
+      IDS_AUTOFILL_DIALOG_ACCEPT_TERMS, ConfirmButtonText());

[Evan Stade, 2013/02/08 15:38:53]

Just make confirmbuttontext part of the translation. I don't think this approach
internationalizes well.

[Dan Beam, 2013/02/08 20:53:06]

Done.

+  TrimWhitespace(full_translation, TRIM_ALL, &full_translation);
+
+  std::vector<string16> sentences;
+  base::SplitString(full_translation, '\n', &sentences);
+  DCHECK_EQ(2U, sentences.size());
+
+  std::vector<string16> parts;
+  base::SplitStringDontTrim(sentences.front(), '|', &parts);
+  DCHECK_EQ(5U, parts.size());
+
+  return std::make_pair(parts, sentences.back());

[Evan Stade, 2013/02/08 15:38:53]

Instead of front and back, use operator[]

[Dan Beam, 2013/02/08 20:53:06]

Done. (but why?)

+}
+
+bool AutofillDialogControllerImpl::IsFootnoteShowing() const {
+  return HasRequiredAction(wallet::ACCEPT_TOS);
+}
+
 DialogSignedInState AutofillDialogControllerImpl::SignedInState() const {
   if (!wallet_items_)
     return REQUIRES_RESPONSE;
 
   if (HasRequiredAction(wallet::GAIA_AUTH))
     return REQUIRES_SIGN_IN;
 
   if (HasRequiredAction(wallet::PASSIVE_GAIA_AUTH))
     return REQUIRES_PASSIVE_SIGN_IN;
 
   return SIGNED_IN;
 }
 
-string16 AutofillDialogControllerImpl::SaveLocallyText() const {
-  return l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_SAVE_LOCALLY_CHECKBOX);
-}
+DialogNotification AutofillDialogControllerImpl::CurrentNotification() const {
+  if (HasRequiredAction(wallet::VERIFY_CVV)) {
+    return DialogNotification(
+        DialogNotification::REQUIRED_ACTION,
+        l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_VERIFY_CVV));
+  }
 
-string16 AutofillDialogControllerImpl::ProgressBarText() const {
-  return l10n_util::GetStringUTF16(
-      IDS_AUTOFILL_DIALOG_AUTOCHECKOUT_PROGRESS_BAR);
+  if (RequestingCreditCardInfo() && !TransmissionWillBeSecure()) {
+    return DialogNotification(
+        DialogNotification::SECURITY_WARNING,
+        l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_SECURITY_WARNING));
+  }
+
+  if (!invoked_from_same_origin_) {
+    return DialogNotification(
+        DialogNotification::SECURITY_WARNING,
+        l10n_util::GetStringFUTF16(
+            IDS_AUTOFILL_DIALOG_SITE_WARNING, UTF8ToUTF16(source_url_.host())));
+  }
+
+  return DialogNotification();
 }
 
 const DetailInputs& AutofillDialogControllerImpl::RequestedFieldsForSection(
     DialogSection section) const {
   switch (section) {
     case SECTION_EMAIL:
       return requested_email_fields_;
     case SECTION_CC:
       return requested_cc_fields_;
     case SECTION_BILLING:
@@ skipping 204 matching lines @@
                           popup_labels,
                           popup_icons,
                           popup_ids);
   section_showing_popup_ = section;
 }
 
 void AutofillDialogControllerImpl::FocusMoved() {
   HidePopup();
 }
 
+Profile* AutofillDialogControllerImpl::profile() {
+  return profile_;
+}
+
+content::WebContents* AutofillDialogControllerImpl::web_contents() {
+  return contents_;
+}
+
+void AutofillDialogControllerImpl::StartSignInFlow() {
+  DCHECK(registrar_.IsEmpty());
+
+  content::Source<content::NavigationController> source(
+      &view_->ShowSignIn());
+  registrar_.Add(this, content::NOTIFICATION_NAV_ENTRY_COMMITTED, source);
+}
+
+void AutofillDialogControllerImpl::EndSignInFlow() {
+  DCHECK(!registrar_.IsEmpty());
+  registrar_.RemoveAll();
+  view_->HideSignIn();
+}
+
+void AutofillDialogControllerImpl::ShowTermsOfService() {
+  OpenInNewTab(GURL(kTermsOfServiceUrl));
+}
+
+void AutofillDialogControllerImpl::ShowPrivacyPolicy() {
+  OpenInNewTab(GURL(kPrivacyPolicyUrl));
+}
+
 void AutofillDialogControllerImpl::ViewClosed(DialogAction action) {
   if (action == ACTION_SUBMIT) {
     FillOutputForSection(SECTION_EMAIL);
     FillOutputForSection(SECTION_CC);
     FillOutputForSection(SECTION_BILLING);
     if (view_->UseBillingForShipping()) {
       FillOutputForSectionWithComparator(
           SECTION_BILLING,
           base::Bind(DetailInputMatchesShippingField));
       FillOutputForSectionWithComparator(
           SECTION_CC,
           base::Bind(DetailInputMatchesShippingField));
     } else {
       FillOutputForSection(SECTION_SHIPPING);
     }
     callback_.Run(&form_structure_);
   } else {
     callback_.Run(NULL);
   }
 
   delete this;
 }
 
-void AutofillDialogControllerImpl::UpdateProgressBar(double value) {
-  view_->UpdateProgressBar(value);
-}
-
-DialogNotification AutofillDialogControllerImpl::CurrentNotification() const {
-  if (HasRequiredAction(wallet::VERIFY_CVV)) {
-    return DialogNotification(
-        DialogNotification::REQUIRED_ACTION,
-        l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_VERIFY_CVV));
-  }
-
-  if (RequestingCreditCardInfo() && !TransmissionWillBeSecure()) {
-    return DialogNotification(
-        DialogNotification::SECURITY_WARNING,
-        l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_SECURITY_WARNING));
-  }
-
-  if (!invoked_from_same_origin_) {
-    return DialogNotification(
-        DialogNotification::SECURITY_WARNING,
-        l10n_util::GetStringFUTF16(
-            IDS_AUTOFILL_DIALOG_SITE_WARNING, UTF8ToUTF16(source_url_.host())));
-  }
-
-  return DialogNotification();
-}
-
-void AutofillDialogControllerImpl::StartSignInFlow() {
-  DCHECK(registrar_.IsEmpty());
-
-  content::Source<content::NavigationController> source(
-      &view_->ShowSignIn());
-  registrar_.Add(this, content::NOTIFICATION_NAV_ENTRY_COMMITTED, source);
-}
-
-void AutofillDialogControllerImpl::EndSignInFlow() {
-  DCHECK(!registrar_.IsEmpty());
-  registrar_.RemoveAll();
-  view_->HideSignIn();
-}
-
-Profile* AutofillDialogControllerImpl::profile() {
-  return profile_;
-}
-
-content::WebContents* AutofillDialogControllerImpl::web_contents() {
-  return contents_;
-}
-
 ////////////////////////////////////////////////////////////////////////////////
 // AutofillPopupDelegate
 
 void AutofillDialogControllerImpl::OnPopupShown(
     content::KeyboardListener* listener) {}
 
 void AutofillDialogControllerImpl::OnPopupHidden(
     content::KeyboardListener* listener) {}
 
 void AutofillDialogControllerImpl::DidSelectSuggestion(int identifier) {
@@ skipping 81 matching lines @@
 void AutofillDialogControllerImpl::OnDidGetFullWallet(
     scoped_ptr<wallet::FullWallet> full_wallet) {
   NOTIMPLEMENTED();
 }
 
 void AutofillDialogControllerImpl::OnDidGetWalletItems(
     scoped_ptr<wallet::WalletItems> wallet_items) {
   wallet_items_ = wallet_items.Pass();
   view_->UpdateAccountChooser();
   view_->UpdateNotificationArea();
+  view_->UpdateFootnote();
 }
 
 void AutofillDialogControllerImpl::OnDidSaveAddress(
     const std::string& address_id) {
   NOTIMPLEMENTED() << " address_id=" << address_id;
 }
 
 void AutofillDialogControllerImpl::OnDidSaveInstrument(
     const std::string& instrument_id) {
   NOTIMPLEMENTED() << " instrument_id=" << instrument_id;
@@ skipping 19 matching lines @@
   wallet_items_.reset();
 }
 
 void AutofillDialogControllerImpl::OnNetworkError(int response_code) {
   NOTIMPLEMENTED() << " response_code=" << response_code;
   wallet_items_.reset();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 
+void AutofillDialogControllerImpl::UpdateProgressBar(double value) {
+  view_->UpdateProgressBar(value);
+}
+
 bool AutofillDialogControllerImpl::HandleKeyPressEventInInput(
     const content::NativeWebKeyboardEvent& event) {
   if (popup_controller_)
     return popup_controller_->HandleKeyPressEvent(event);
 
   return false;
 }
 
 bool AutofillDialogControllerImpl::RequestingCreditCardInfo() const {
   DCHECK_GT(form_structure_.field_count(), 0U);
@@ skipping 208 matching lines @@
 DetailInputs* AutofillDialogControllerImpl::MutableRequestedFieldsForSection(
     DialogSection section) {
   return const_cast<DetailInputs*>(&RequestedFieldsForSection(section));
 }
 
 void AutofillDialogControllerImpl::HidePopup() {
   if (popup_controller_)
     popup_controller_->Hide();
 }
 
+void AutofillDialogControllerImpl::OpenInNewTab(const GURL& url) {
+  Browser* browser = chrome::FindOrCreateTabbedBrowser(
+      profile_, chrome::GetActiveDesktop());

[Evan Stade, 2013/02/08 15:38:53]

You want to get the browser from the web contents (via browserfinder)

[Dan Beam, 2013/02/08 20:53:06]

Done. Btw, can this API ever be triggered by a non-tabbed browser?

[Evan Stade, 2013/02/11 01:05:23]

Potentially. It could be used in a popup browser window, extension popup (not
sure if those support constrained windows), or an app. I don't think we
explicitly disable this stuff, or want to.

[Dan Beam, 2013/02/11 16:45:18]

Why wouldn't we want to find or created a *tabbed* browser, then, rather than
just finding the browser with the current web contents?

[Evan Stade, 2013/02/12 01:36:54]

Well, the code you currently have might add a new tab in the wrong window, which
is not acceptable.

+  chrome::NavigateParams params(browser, url, content::PAGE_TRANSITION_LINK);
+  params.disposition = NEW_FOREGROUND_TAB;
+  chrome::Navigate(&params);
+}
+
 }  // namespace autofill