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

chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer_browsertest.cc

Index: chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer_browsertest.cc
diff --git a/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer_browsertest.cc b/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer_browsertest.cc
index 01535a44e2a68383550108db0c3e52cf79810967..0fc67d5abc02bb061f0a2fd125afc247b2c1a515 100644
--- a/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer_browsertest.cc
+++ b/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer_browsertest.cc
@@ -7,6 +7,9 @@
 #include "base/command_line.h"
 #include "base/macros.h"
 #include "base/test/histogram_tester.h"
+#include "base/test/scoped_feature_list.h"
+#include "chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.h"

[Ilya Sherman, 2017/06/28 19:56:05]

nit: Is this #include needed as part of this CL?

[chaopeng, 2017/06/30 19:32:36]

Done.

+#include "chrome/common/chrome_features.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/common/content_switches.h"
@@ -16,21 +19,23 @@
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
-const char kAdsMetricsFeature[] = "AdsMetrics";
-
 class AdsPageLoadMetricsObserverBrowserTest : public InProcessBrowserTest {
  public:
-  AdsPageLoadMetricsObserverBrowserTest() {
-    base::CommandLine* cmd_line = base::CommandLine::ForCurrentProcess();
-    cmd_line->AppendSwitchASCII(switches::kEnableFeatures, kAdsMetricsFeature);
-  }
+  AdsPageLoadMetricsObserverBrowserTest() {}
+
   ~AdsPageLoadMetricsObserverBrowserTest() override {}
 
+  void SetUp() override {
+    scoped_feature_list_.InitAndEnableFeature(features::kAdsFeature);
+    InProcessBrowserTest::SetUp();
+  }
+
   void SetUpOnMainThread() override {
     ASSERT_TRUE(embedded_test_server()->Start());
   }
 
  private:
+  base::test::ScopedFeatureList scoped_feature_list_;
   DISALLOW_COPY_AND_ASSIGN(AdsPageLoadMetricsObserverBrowserTest);
 };