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

Unified Diff: components/autofill/content/browser/autocheckout_manager.cc

Issue 21054003: Autocheckout: Manually revert r208999 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: components/autofill/content/browser/autocheckout_manager.cc
diff --git a/components/autofill/content/browser/autocheckout_manager.cc b/components/autofill/content/browser/autocheckout_manager.cc
index ae788ce7af44a0e447670d486971e8a27f954e97..7f7304f45468887a43ecb666ee93de99880f2fa4 100644
--- a/components/autofill/content/browser/autocheckout_manager.cc
+++ b/components/autofill/content/browser/autocheckout_manager.cc
@@ -8,7 +8,6 @@
#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"
@@ -198,8 +197,6 @@ void AutocheckoutManager::FillForms() {
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(
@@ -340,8 +337,6 @@ void AutocheckoutManager::ReturnAutocheckoutData(
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;
@@ -503,7 +498,6 @@ void AutocheckoutManager::SendAutocheckoutStatus(AutocheckoutStatus status) {
autocheckout_request_manager->SendAutocheckoutStatus(
status,
autofill_manager_->GetWebContents()->GetURL(),
- latency_statistics_,
google_transaction_id_);
// Log the result of this Autocheckout flow to UMA.
@@ -524,23 +518,6 @@ void AutocheckoutManager::SetStepProgressForPage(
}
}
-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_);

Powered by Google App Engine
This is Rietveld 408576698