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

chrome/browser/apps/guest_view/web_view_browsertest.cc

 // 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.
 
 #include <queue>
 #include <set>
 #include <utility>
 
 #include "base/callback_helpers.h"
 #include "base/files/file_util.h"
@@ skipping 3605 matching lines @@
     if (use_cross_process_frames_for_guests) {
       scoped_feature_list_.InitAndEnableFeature(
           features::kGuestViewCrossProcessFrames);
     } else {
       scoped_feature_list_.InitAndDisableFeature(
           features::kGuestViewCrossProcessFrames);
     }
   }
 
  private:
+  base::test::ScopedFeatureList scoped_feature_list_;
   DISALLOW_COPY_AND_ASSIGN(WebViewGuestScrollTest);
-
-  base::test::ScopedFeatureList scoped_feature_list_;

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

This change seems unrelated.  Could you please split it off to a separate CL?

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

Removed

 };
 
 class WebViewGuestScrollTouchTest : public WebViewGuestScrollTest {
  protected:
   void SetUpCommandLine(base::CommandLine* command_line) override {
     WebViewGuestScrollTest::SetUpCommandLine(command_line);
 
     command_line->AppendSwitchASCII(
         switches::kTouchEventFeatureDetection,
         switches::kTouchEventFeatureDetectionEnabled);
@@ skipping 546 matching lines @@
                        ClosingChromeSignInShouldNotCrash) {
   GURL signin_url{"chrome://chrome-signin"};
 
   AddTabAtIndex(0, signin_url, ui::PAGE_TRANSITION_TYPED);
   AddTabAtIndex(1, signin_url, ui::PAGE_TRANSITION_TYPED);
   WaitForWebViewInDom();
 
   chrome::CloseTab(browser());
 }
 #endif