Autocheckout: Manually revert r208999

components/autofill/content/browser/autocheckout_manager.cc

 // Copyright 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/content/browser/autocheckout_manager.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/content/browser/autocheckout_request_manager.h"
-#include "components/autofill/content/browser/autocheckout_statistic.h"
 #include "components/autofill/content/browser/autocheckout_steps.h"
 #include "components/autofill/core/browser/autofill_country.h"
 #include "components/autofill/core/browser/autofill_field.h"
 #include "components/autofill/core/browser/autofill_manager.h"
 #include "components/autofill/core/browser/autofill_metrics.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/autofill/core/browser/field_types.h"
 #include "components/autofill/core/browser/form_structure.h"
 #include "components/autofill/core/common/autofill_messages.h"
@@ skipping 169 matching lines @@
       autofill_manager_->GetWebContents()->GetRenderViewHost();
   if (!host)
     return;
 
   host->Send(new AutofillMsg_FillFormsAndClick(
       host->GetRoutingID(),
       filled_forms,
       page_meta_data_->click_elements_before_form_fill,
       page_meta_data_->click_elements_after_form_fill,
       page_meta_data_->proceed_element_descriptor));
-  // Record time taken for navigating current page.
-  RecordTimeTaken(page_meta_data_->current_page_number);
 }
 
 void AutocheckoutManager::OnAutocheckoutPageCompleted(
     AutocheckoutStatus status) {
   if (!in_autocheckout_flow_)
     return;
 
   DVLOG(2) << "OnAutocheckoutPageCompleted, page_no: "
            << page_meta_data_->current_page_number
            << " status: "
@@ skipping 120 matching lines @@
     const FormStructure* result,
     const std::string& google_transaction_id) {
   if (!result) {
     // When user cancels the dialog, |result| is NULL.
     // TODO(): add AutocheckoutStatus.USER_CANCELLED, and call
     //         EndAutocheckout(USER_CANCELLED) instead.
     in_autocheckout_flow_ = false;
     return;
   }
 
-  latency_statistics_.clear();
-  last_step_completion_timestamp_ = base::TimeTicks().Now();
   google_transaction_id_ = google_transaction_id;
   in_autocheckout_flow_ = true;
   should_preserve_dialog_ = true;
   metric_logger_->LogAutocheckoutBuyFlowMetric(
       AutofillMetrics::AUTOCHECKOUT_BUY_FLOW_STARTED);
 
   profile_.reset(new AutofillProfile());
   credit_card_.reset(new CreditCard());
   billing_address_.reset(new AutofillProfile());
 
@@ skipping 141 matching lines @@
   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(),
-      latency_statistics_,
       google_transaction_id_);
 
   // Log the result of this Autocheckout flow to UMA.
   metric_logger_->LogAutocheckoutBuyFlowMetric(
       AutocheckoutStatusToUmaMetric(status));
 
   google_transaction_id_ = kTransactionIdNotSet;
 }
 
 void AutocheckoutManager::SetStepProgressForPage(
     int page_number,
     AutocheckoutStepStatus status) {
   if (page_types_.count(page_number) == 1) {
     for (size_t i = 0; i < page_types_[page_number].size(); ++i) {
       autofill_manager_->delegate()->UpdateAutocheckoutStep(
           page_types_[page_number][i], status);
     }
   }
 }
 
-void AutocheckoutManager::RecordTimeTaken(int page_number) {
-  AutocheckoutStatistic statistic;
-  statistic.page_number = page_number;
-  if (page_types_.count(page_number) == 1) {
-    for (size_t i = 0; i < page_types_[page_number].size(); ++i) {
-      statistic.steps.push_back(page_types_[page_number][i]);
-    }
-  }
-
-  statistic.time_taken =
-      base::TimeTicks().Now() - last_step_completion_timestamp_;
-  latency_statistics_.push_back(statistic);
-
-  // Reset timestamp.
-  last_step_completion_timestamp_ = base::TimeTicks().Now();
-}
-
 void AutocheckoutManager::EndAutocheckout(AutocheckoutStatus status) {
   DCHECK(in_autocheckout_flow_);
 
   DVLOG(2) << "EndAutocheckout at step: "
            << page_meta_data_->current_page_number
            << " with status: "
            << status;
 
   SendAutocheckoutStatus(status);
   if (status == SUCCESS)
     autofill_manager_->delegate()->OnAutocheckoutSuccess();
   else
     autofill_manager_->delegate()->OnAutocheckoutError();
   in_autocheckout_flow_ = false;
 }
 
 }  // namespace autofill