Disable AutoImport: experiment and histogram for analysis.

chrome/common/chrome_features.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file defines all the public base::FeatureList features for the chrome
 // module.
 
 #ifndef CHROME_COMMON_CHROME_FEATURES_H_
 #define CHROME_COMMON_CHROME_FEATURES_H_
 
@@ skipping 17 matching lines @@
 #endif  // defined(OS_WIN) || defined(OS_LINUX)
 
 extern const base::Feature kBackspaceGoesBackFeature;
 
 extern const base::Feature kBlockPromptsIfDismissedOften;
 
 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;
 
 extern const base::Feature kExperimentalKeyboardLockUI;
 
 #if defined(OS_CHROMEOS)
 extern const base::Feature kHappinessTrackingSystem;
 #endif
 
@@ skipping 32 matching lines @@
 
 extern const base::Feature kQuickUnlockPin;
 #endif  // defined(OS_CHROMEOS)
 
 // DON'T ADD RANDOM STUFF HERE. Put it in the main section above in
 // alphabetical order, or in one of the ifdefs (also in order in each section).
 
 }  // namespace features
 
 #endif  // CHROME_COMMON_CHROME_FEATURES_H_