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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ios/chrome/browser/autofill/personal_data_manager_factory.cc

Issue 2740633002: [Autofill] Add upstreaming UKM (Closed)
Patch Set: Created 3 years, 9 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
« components/autofill/core/browser/personal_data_manager.cc ('K') | « components/ukm/ukm_service.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/autofill/personal_data_manager_factory.cc
diff --git a/ios/chrome/browser/autofill/personal_data_manager_factory.cc b/ios/chrome/browser/autofill/personal_data_manager_factory.cc
index 4e0a83999cea45f377794f0bf284aac7ab308f98..78dad524a77eb19b59c6fa6d6a3b7b4e85c685e7 100644
--- a/ios/chrome/browser/autofill/personal_data_manager_factory.cc
+++ b/ios/chrome/browser/autofill/personal_data_manager_factory.cc
@@ -50,7 +50,8 @@ PersonalDataManagerFactory::BuildServiceInstanceFor(
ios::ChromeBrowserState* chrome_browser_state =
ios::ChromeBrowserState::FromBrowserState(context);
std::unique_ptr<PersonalDataManager> service(
- new PersonalDataManager(GetApplicationContext()->GetApplicationLocale()));
+ new PersonalDataManager(GetApplicationContext()->GetApplicationLocale(),
+ nullptr /* ukm::UkmService */));
Mathieu 2017/03/08 22:33:49 is there no ukm service on iOS?
service->Init(
ios::WebDataServiceFactory::GetAutofillWebDataForBrowserState(
chrome_browser_state, ServiceAccessType::EXPLICIT_ACCESS),
« components/autofill/core/browser/personal_data_manager.cc ('K') | « components/ukm/ukm_service.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698