The logging of Autofill UKMs is hidden behing a Feature. The objective of that Feature was to act as a killswitch, but this is now handled directly with UKMs.
Issue 721787 has been merged into this issue.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/06fc093968064421daf828caf503ef5fafb07cba commit 06fc093968064421daf828caf503ef5fafb07cba Author: sebsg <sebsg@chromium.org> Date: Tue May 16 14:07:46 2017 [Autofill] Enable Autofill UKM by default. BUG= 718014 Review-Url: https://codereview.chromium.org/2874413004 Cr-Commit-Position: refs/heads/master@{#472094} [modify] https://crrev.com/06fc093968064421daf828caf503ef5fafb07cba/components/autofill/core/browser/autofill_experiments.cc [modify] https://crrev.com/06fc093968064421daf828caf503ef5fafb07cba/components/autofill/core/browser/autofill_experiments.h [modify] https://crrev.com/06fc093968064421daf828caf503ef5fafb07cba/components/autofill/core/browser/autofill_manager_unittest.cc [modify] https://crrev.com/06fc093968064421daf828caf503ef5fafb07cba/components/autofill/core/browser/autofill_metrics.cc [modify] https://crrev.com/06fc093968064421daf828caf503ef5fafb07cba/components/autofill/core/browser/autofill_metrics_unittest.cc [modify] https://crrev.com/06fc093968064421daf828caf503ef5fafb07cba/components/payments/core/journey_logger.cc [modify] https://crrev.com/06fc093968064421daf828caf503ef5fafb07cba/components/payments/core/journey_logger_unittest.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3573e1fcb10702a8d842cef0ca9ff2825a4c85db commit 3573e1fcb10702a8d842cef0ca9ff2825a4c85db Author: dmazzoni <dmazzoni@chromium.org> Date: Tue May 16 18:50:28 2017 Revert of [Autofill] Enable Autofill UKM by default. (patchset #4 id:60001 of https://codereview.chromium.org/2874413004/ ) Reason for revert: Failing consistently on chromium.memory/Linux ASan LSan Tests (1) See crbug.com/722947 Original issue's description: > [Autofill] Enable Autofill UKM by default. > > BUG= 718014 > > Review-Url: https://codereview.chromium.org/2874413004 > Cr-Commit-Position: refs/heads/master@{#472094} > Committed: https://chromium.googlesource.com/chromium/src/+/06fc093968064421daf828caf503ef5fafb07cba TBR=csashi@google.com,mathp@chromium.org,sebsg@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= 718014 Review-Url: https://codereview.chromium.org/2888633002 Cr-Commit-Position: refs/heads/master@{#472178} [modify] https://crrev.com/3573e1fcb10702a8d842cef0ca9ff2825a4c85db/components/autofill/core/browser/autofill_experiments.cc [modify] https://crrev.com/3573e1fcb10702a8d842cef0ca9ff2825a4c85db/components/autofill/core/browser/autofill_experiments.h [modify] https://crrev.com/3573e1fcb10702a8d842cef0ca9ff2825a4c85db/components/autofill/core/browser/autofill_manager_unittest.cc [modify] https://crrev.com/3573e1fcb10702a8d842cef0ca9ff2825a4c85db/components/autofill/core/browser/autofill_metrics.cc [modify] https://crrev.com/3573e1fcb10702a8d842cef0ca9ff2825a4c85db/components/autofill/core/browser/autofill_metrics_unittest.cc [modify] https://crrev.com/3573e1fcb10702a8d842cef0ca9ff2825a4c85db/components/payments/core/journey_logger.cc [modify] https://crrev.com/3573e1fcb10702a8d842cef0ca9ff2825a4c85db/components/payments/core/journey_logger_unittest.cc
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f9df7482eb3b4ae01667614195c8878c74b9ae6a commit f9df7482eb3b4ae01667614195c8878c74b9ae6a Author: sebsg <sebsg@chromium.org> Date: Fri May 19 22:21:53 2017 Reland - [Autofill] Enable Autofill UKM by default. Initial CL: https://codereview.chromium.org/2874413004/ Was reverted because it triggered a DCHECK in the UKMFormLogger. csashi@ fixed this issue now (see crbug.com/722947 ). BUG= 718014 Review-Url: https://codereview.chromium.org/2897573003 Cr-Commit-Position: refs/heads/master@{#473362} [modify] https://crrev.com/f9df7482eb3b4ae01667614195c8878c74b9ae6a/components/autofill/core/browser/autofill_experiments.cc [modify] https://crrev.com/f9df7482eb3b4ae01667614195c8878c74b9ae6a/components/autofill/core/browser/autofill_experiments.h [modify] https://crrev.com/f9df7482eb3b4ae01667614195c8878c74b9ae6a/components/autofill/core/browser/autofill_manager_unittest.cc [modify] https://crrev.com/f9df7482eb3b4ae01667614195c8878c74b9ae6a/components/autofill/core/browser/autofill_metrics.cc [modify] https://crrev.com/f9df7482eb3b4ae01667614195c8878c74b9ae6a/components/autofill/core/browser/autofill_metrics_unittest.cc [modify] https://crrev.com/f9df7482eb3b4ae01667614195c8878c74b9ae6a/components/payments/core/journey_logger.cc [modify] https://crrev.com/f9df7482eb3b4ae01667614195c8878c74b9ae6a/components/payments/core/journey_logger_unittest.cc
Comment 1 by se...@chromium.org
, May 12 2017