Disable AutoImport: experiment and histogram for analysis.

chrome/common/chrome_features.h

Index: chrome/common/chrome_features.h
diff --git a/chrome/common/chrome_features.h b/chrome/common/chrome_features.h
index 5f3a814089a90bb24feb340745cdecfb1dce6f4a..26641e31e246a803498fd28ee2f5bcab7f6bffba 100644
--- a/chrome/common/chrome_features.h
+++ b/chrome/common/chrome_features.h
@@ -35,6 +35,11 @@ extern const base::Feature kBlockSmallContent;
 
 extern const base::Feature kBrowserHangFixesExperiment;
 
+
+#if defined(OS_WIN)
+extern const base::Feature kDisableFirstRunAutoImportWin;

[sky, 2016/08/29 16:08:56]

If possible, use Enable instead of disable. That way code looks like:

if (isEnabled...

vs

if (!isDisabled), which is easier to get wrong.

[gcomanici, 2016/08/30 18:30:30]

I think the name should stay kDisableFirstRunAutoImportWin to avoid confusion in
the list of features. The default behavior is to run Auto Import on a first
browser run, so the feature would disable it. Also, code in
chrome_browser_main.cc indeed has the line

if(IsEnabled(features::kDisableFirstRunAutoImportWin)).

+#endif  // defined(OS_WIN)
+
 extern const base::Feature kDisplayPersistenceToggleInPermissionPrompts;
 
 extern const base::Feature kExpectCTReporting;