[Cleanup] Move all browsertests to use ScopedFeatureList to modify features

chrome/browser/net/predictor_tab_helper.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_NET_PREDICTOR_TAB_HELPER_H_
 #define CHROME_BROWSER_NET_PREDICTOR_TAB_HELPER_H_
 
+#include "base/feature_list.h"
 #include "base/macros.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/reload_type.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
 
+namespace features {
+
+extern const base::Feature kPreconnectMore;
+
+}  // namespace features

[Ilya Sherman, 2017/06/12 21:50:29]

I don't think that it's appropriate for this header to add to the features
namespace.  Instead, please declare and define this feature in
//chrome/common/chrome_features.{h,cc}.

[chaopeng, 2017/06/28 19:21:24]

Done.

+
 namespace content {
 class NavigationHandle;
 }
 
 namespace chrome_browser_net {
 
 class PredictorTabHelper
     : public content::WebContentsObserver,
       public content::WebContentsUserData<PredictorTabHelper> {
  public:
@@ skipping 21 matching lines @@
   // render process spawn (pre-PlzNavigate), which can take substantial time
   // especially on Android.
   bool predicted_from_pending_entry_;
 
   DISALLOW_COPY_AND_ASSIGN(PredictorTabHelper);
 };
 
 }  // namespace chrome_browser_net
 
 #endif  // CHROME_BROWSER_NET_PREDICTOR_TAB_HELPER_H_