Implements SendAutocheckoutStatus API calls for stats tracking.

components/autofill/browser/autocheckout_manager.cc

 // Copyright (c) 2013 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 "components/autofill/browser/autocheckout_manager.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/utf_string_conversions.h"
+#include "components/autofill/browser/autocheckout_request_manager.h"
 #include "components/autofill/browser/autofill_country.h"
 #include "components/autofill/browser/autofill_field.h"
 #include "components/autofill/browser/autofill_manager.h"
 #include "components/autofill/browser/autofill_profile.h"
 #include "components/autofill/browser/credit_card.h"
 #include "components/autofill/browser/field_types.h"
 #include "components/autofill/browser/form_structure.h"
-#include "components/autofill/common/autocheckout_status.h"
 #include "components/autofill/common/autofill_messages.h"
 #include "components/autofill/common/form_data.h"
 #include "components/autofill/common/form_field_data.h"
 #include "components/autofill/common/web_element_descriptor.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/ssl_status.h"
 #include "googleurl/src/gurl.h"
 #include "ui/gfx/rect.h"
 
@@ skipping 43 matching lines @@
 }  // namespace
 
 namespace autofill {
 
 AutocheckoutManager::AutocheckoutManager(AutofillManager* autofill_manager)
     : autofill_manager_(autofill_manager),
       autocheckout_offered_(false),
       is_autocheckout_bubble_showing_(false),
       in_autocheckout_flow_(false),
       ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)) {
+  AutocheckoutRequestManager::CreateFromBrowserContext(
+      autofill_manager_->GetWebContents()->GetBrowserContext());
 }
 
 AutocheckoutManager::~AutocheckoutManager() {
 }
 
 void AutocheckoutManager::FillForms() {
   // |page_meta_data_| should have been set by OnLoadedPageMetaData.
   DCHECK(page_meta_data_);
 
   // Fill the forms on the page with data given by user.
@@ skipping 19 matching lines @@
       autofill_manager_->GetWebContents()->GetRenderViewHost();
   if (!host)
     return;
 
   host->Send(new AutofillMsg_FillFormsAndClick(
       host->GetRoutingID(),
       filled_forms,
       *page_meta_data_->proceed_element_descriptor));
 }
 
+void AutocheckoutManager::OnClickFailed(AutocheckoutStatus status) {
+  AutocheckoutRequestManager* autocheckout_request_manager =
+      AutocheckoutRequestManager::FromBrowserContext(
+          autofill_manager_->GetWebContents()->GetBrowserContext());
+  // It is assumed that the domain Autocheckout starts on does not change during
+  // the flow.  If this proves to be incorrect, the |source_url| from
+  // AutofillDialogControllerImpl will need to be provided in its callback in
+  // addition to the Google transaction id.
+  autocheckout_request_manager->SendAutocheckoutStatus(
+      status,
+      autofill_manager_->GetWebContents()->GetURL(),
+      google_transaction_id_);

[Ilya Sherman, 2013/03/26 23:50:01]

nit: DCHECK that the transaction_id_ is set?

[ahutter, 2013/03/27 01:23:33]

By set do you mean non-empty? Because empty is ok.  That's what will happen if
the user chooses to Autocheckout with local data.

[Ilya Sherman, 2013/03/27 01:36:55]

Maybe you should initialize the transaction id to some arbitrary string constant
kTransactionIdNotSet, and DCHECK that was changed away from that?  It just seems
good to make sure that you're not reading stale data.

[ahutter, 2013/03/27 19:30:49]

Done.

+  autofill_manager_->delegate()->OnAutocheckoutError();
+}
+
+
 void AutocheckoutManager::OnLoadedPageMetaData(
     scoped_ptr<AutocheckoutPageMetaData> page_meta_data) {
   scoped_ptr<AutocheckoutPageMetaData> old_meta_data =
       page_meta_data_.Pass();
   page_meta_data_ = page_meta_data.Pass();
 
   // On the first page of an Autocheckout flow, when this function is called the
   // user won't have opted into the flow yet.
   if (!in_autocheckout_flow_)
     return;
 
+  AutocheckoutStatus status = SUCCESS;
+
   // Missing Autofill server results.
   if (!page_meta_data_) {
     in_autocheckout_flow_ = false;
+    status = MISSING_FIELDMAPPING;
   } else if (page_meta_data_->IsStartOfAutofillableFlow()) {
     // Not possible unless Autocheckout failed to proceed.
     in_autocheckout_flow_ = false;
+    status = CANNOT_PROCEED;
   } else if (!page_meta_data_->IsInAutofillableFlow()) {
     // Missing Autocheckout meta data in the Autofill server results.
     in_autocheckout_flow_ = false;
+    status = MISSING_FIELDMAPPING;
   } else if (page_meta_data_->current_page_number <=
                  old_meta_data->current_page_number) {
     // Not possible unless Autocheckout failed to proceed.
     in_autocheckout_flow_ = false;
+    status = CANNOT_PROCEED;
   }
 
   // Encountered an error during the Autocheckout flow.
   if (!in_autocheckout_flow_) {
-    // TODO(ahutter): SendAutocheckoutStatus of the error.
+    AutocheckoutRequestManager* autocheckout_request_manager =
+        AutocheckoutRequestManager::FromBrowserContext(
+            autofill_manager_->GetWebContents()->GetBrowserContext());
+    // It is assumed that the domain Autocheckout starts on does not change
+    // during the flow.  If this proves to be incorrect, the |source_url| from
+    // AutofillDialogControllerImpl will need to be provided in its callback in
+    // addition to the Google transaction id.
+    autocheckout_request_manager->SendAutocheckoutStatus(
+        status,
+        autofill_manager_->GetWebContents()->GetURL(),
+        google_transaction_id_);

[Ilya Sherman, 2013/03/26 23:50:01]

nit: Decompose this code into a shared method rather than repeating it.

[ahutter, 2013/03/27 01:23:33]

Done.

     autofill_manager_->delegate()->OnAutocheckoutError();
     return;
   }
 
   // Add 1.0 since page numbers are 0-indexed.
   autofill_manager_->delegate()->UpdateProgressBar(
       (1.0 + page_meta_data_->current_page_number) /
           page_meta_data_->total_pages);
   FillForms();
   // If the current page is the last page in the flow, close the dialog.
   if (page_meta_data_->IsEndOfAutofillableFlow()) {
-    // TODO(ahutter): SendAutocheckoutStatus of SUCCESS.
+    AutocheckoutRequestManager* autocheckout_request_manager =
+        AutocheckoutRequestManager::FromBrowserContext(
+            autofill_manager_->GetWebContents()->GetBrowserContext());
+    // It is assumed that the domain Autocheckout starts on does not change
+    // during the flow.  If this proves to be incorrect, the |source_url| from
+    // AutofillDialogControllerImpl will need to be provided in its callback in
+    // addition to the Google transaction id.
+    autocheckout_request_manager->SendAutocheckoutStatus(
+        status,
+        autofill_manager_->GetWebContents()->GetURL(),
+        google_transaction_id_);

[Ilya Sherman, 2013/03/26 23:50:01]

Ditto.

[ahutter, 2013/03/27 01:23:33]

Done.

     autofill_manager_->delegate()->HideRequestAutocompleteDialog();
     in_autocheckout_flow_ = false;
   }
 }
 
 void AutocheckoutManager::OnFormsSeen() {
   autocheckout_offered_ = false;
 }
 
 void AutocheckoutManager::MaybeShowAutocheckoutBubble(
@@ skipping 22 matching lines @@
 void AutocheckoutManager::MaybeShowAutocheckoutDialog(
     const GURL& frame_url,
     const SSLStatus& ssl_status,
     bool show_dialog) {
   is_autocheckout_bubble_showing_ = false;
   if (!show_dialog)
     return;
 
   FormData form = BuildAutocheckoutFormData();
   form.ssl_status = ssl_status;
-  base::Callback<void(const FormStructure*)> callback =
+  base::Callback<void(const FormStructure*, const std::string&)> callback =
       base::Bind(&AutocheckoutManager::ReturnAutocheckoutData,
                  weak_ptr_factory_.GetWeakPtr());
   autofill_manager_->ShowRequestAutocompleteDialog(
       form, frame_url, DIALOG_TYPE_AUTOCHECKOUT, callback);
 }
 
 bool AutocheckoutManager::IsStartOfAutofillableFlow() const {
   return page_meta_data_ && page_meta_data_->IsStartOfAutofillableFlow();
 }
 
 bool AutocheckoutManager::IsInAutofillableFlow() const {
   return page_meta_data_ && page_meta_data_->IsInAutofillableFlow();
 }
 
-void AutocheckoutManager::ReturnAutocheckoutData(const FormStructure* result) {
+void AutocheckoutManager::ReturnAutocheckoutData(
+    const FormStructure* result,
+    const std::string& google_transaction_id) {
   if (!result)
     return;
 
+  google_transaction_id_ = google_transaction_id;
   in_autocheckout_flow_ = true;
 
   profile_.reset(new AutofillProfile());
   credit_card_.reset(new CreditCard());
 
   for (size_t i = 0; i < result->field_count(); ++i) {
     AutofillFieldType type = result->field(i)->type();
     if (type == CREDIT_CARD_VERIFICATION_CODE) {
       cvv_ = result->field(i)->value;
       continue;
     }
     if (AutofillType(type).group() == AutofillType::CREDIT_CARD) {
       credit_card_->SetRawInfo(result->field(i)->type(),
                                result->field(i)->value);
     } else {
       profile_->SetRawInfo(result->field(i)->type(), result->field(i)->value);
     }
   }
 
   // Add 1.0 since page numbers are 0-indexed.
   autofill_manager_->delegate()->UpdateProgressBar(
       (1.0 + page_meta_data_->current_page_number) /
           page_meta_data_->total_pages);
   FillForms();
 
   // If the current page is the last page in the flow, close the dialog.
   if (page_meta_data_->IsEndOfAutofillableFlow()) {
-    // TODO(ahutter): SendAutocheckoutStatus of SUCCESS.
+    AutocheckoutRequestManager* autocheckout_request_manager =
+        AutocheckoutRequestManager::FromBrowserContext(
+            autofill_manager_->GetWebContents()->GetBrowserContext());
+    // It is assumed that the domain Autocheckout starts on does not change
+    // during the flow.  If this proves to be incorrect, the |source_url| from
+    // AutofillDialogControllerImpl will need to be provided in its callback in
+    // addition to the Google transaction id.
+    autocheckout_request_manager->SendAutocheckoutStatus(
+        SUCCESS,
+        autofill_manager_->GetWebContents()->GetURL(),
+        google_transaction_id_);

[Ilya Sherman, 2013/03/26 23:50:01]

Ditto

[ahutter, 2013/03/27 01:23:33]

Done.

     autofill_manager_->delegate()->HideRequestAutocompleteDialog();
     in_autocheckout_flow_ = false;
   }
 }
 
 void AutocheckoutManager::SetValue(const AutofillField& field,
                                    FormFieldData* field_to_fill) {
   AutofillFieldType type = field.type();
 
   if (type == FIELD_WITH_DEFAULT_VALUE) {
@@ skipping 24 matching lines @@
 
   // TODO(ramankk): Handle variants in a better fashion, need to distinguish
   // between shipping and billing address.
   if (AutofillType(type).group() == AutofillType::CREDIT_CARD)
     credit_card_->FillFormField(field, 0, field_to_fill);
   else
     profile_->FillFormField(field, 0, field_to_fill);
 }
 
 }  // namespace autofill