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

Side by Side Diff: components/autofill/core/browser/autofill_manager.cc

Issue 2955963002: Update Chrome Upstream flow to reflect new UI mocks (Closed)
Patch Set: Prevent AddNewUiFlagStateToRequestIfExperimentOn test from running on Android Created 3 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "components/autofill/core/browser/autofill_manager.h" 5 #include "components/autofill/core/browser/autofill_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 1308 matching lines...) Expand 10 before | Expand all | Expand 10 after
1319 } else { 1319 } else {
1320 upload_decision_metrics |= GetCVCCardUploadDecisionMetric(); 1320 upload_decision_metrics |= GetCVCCardUploadDecisionMetric();
1321 } 1321 }
1322 } 1322 }
1323 if (upload_decision_metrics) { 1323 if (upload_decision_metrics) {
1324 LogCardUploadDecisions(upload_decision_metrics); 1324 LogCardUploadDecisions(upload_decision_metrics);
1325 pending_upload_request_url_ = GURL(); 1325 pending_upload_request_url_ = GURL();
1326 return; 1326 return;
1327 } 1327 }
1328 1328
1329 if (IsAutofillUpstreamShowNewUiExperimentEnabled()) {
1330 upload_request_.active_experiments.push_back(
1331 kAutofillUpstreamShowNewUi.name);
1332 }
1333
1329 // All required data is available, start the upload process. 1334 // All required data is available, start the upload process.
1330 payments_client_->GetUploadDetails(upload_request_.profiles, 1335 payments_client_->GetUploadDetails(upload_request_.profiles,
1331 upload_request_.active_experiments, 1336 upload_request_.active_experiments,
1332 app_locale_); 1337 app_locale_);
1333 } 1338 }
1334 } 1339 }
1335 1340
1336 AutofillMetrics::CardUploadDecisionMetric 1341 AutofillMetrics::CardUploadDecisionMetric
1337 AutofillManager::GetCVCCardUploadDecisionMetric() const { 1342 AutofillManager::GetCVCCardUploadDecisionMetric() const {
1338 if (found_cvc_field_) 1343 if (found_cvc_field_)
(...skipping 1060 matching lines...) Expand 10 before | Expand all | Expand 10 after
2399 #endif // ENABLE_FORM_DEBUG_DUMP 2404 #endif // ENABLE_FORM_DEBUG_DUMP
2400 2405
2401 void AutofillManager::LogCardUploadDecisions(int upload_decision_metrics) { 2406 void AutofillManager::LogCardUploadDecisions(int upload_decision_metrics) {
2402 AutofillMetrics::LogCardUploadDecisionMetrics(upload_decision_metrics); 2407 AutofillMetrics::LogCardUploadDecisionMetrics(upload_decision_metrics);
2403 AutofillMetrics::LogCardUploadDecisionsUkm(client_->GetUkmRecorder(), 2408 AutofillMetrics::LogCardUploadDecisionsUkm(client_->GetUkmRecorder(),
2404 pending_upload_request_url_, 2409 pending_upload_request_url_,
2405 upload_decision_metrics); 2410 upload_decision_metrics);
2406 } 2411 }
2407 2412
2408 } // namespace autofill 2413 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698