Revert of Background tracing: Tracing scenarios can now enable/disable Blink features, for A/B testing purpos…

content/browser/tracing/background_tracing_config_unittest.cc

 // Copyright 2015 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 "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "content/browser/tracing/background_tracing_config_impl.h"
 #include "content/browser/tracing/background_tracing_rule.h"
@@ skipping 238 matching lines @@
   EXPECT_TRUE(config);
   EXPECT_EQ(config->tracing_mode(), BackgroundTracingConfig::PREEMPTIVE);
   EXPECT_EQ(config->category_preset(), BackgroundTracingConfigImpl::BENCHMARK);
   EXPECT_EQ(config->rules().size(), 2u);
   EXPECT_EQ(RuleToString(config->rules()[0]),
             "{\"rule\":\"MONITOR_AND_DUMP_WHEN_TRIGGER_NAMED\","
             "\"trigger_name\":\"foo1\"}");
   EXPECT_EQ(RuleToString(config->rules()[1]),
             "{\"rule\":\"MONITOR_AND_DUMP_WHEN_TRIGGER_NAMED\","
             "\"trigger_name\":\"foo2\"}");
-
-  config = ReadFromJSONString(
-      "{\"category\":\"BENCHMARK_DEEP\",\"configs\":[{\"rule\":"
-      "\"MONITOR_AND_DUMP_WHEN_TRIGGER_NAMED\",\"trigger_name\":"
-      "\"foo1\"}],\"disable_blink_features\":\"SlowerWeb1,SlowerWeb2\","
-      "\"enable_blink_features\":\"FasterWeb1,FasterWeb2\","
-      "\"mode\":\"PREEMPTIVE_TRACING_MODE\","
-      "\"scenario_name\":\"my_awesome_experiment\"}");
-  EXPECT_EQ(config->enable_blink_features(), "FasterWeb1,FasterWeb2");
-  EXPECT_EQ(config->disable_blink_features(), "SlowerWeb1,SlowerWeb2");
-  EXPECT_EQ(config->scenario_name(), "my_awesome_experiment");
 }
 
 TEST_F(BackgroundTracingConfigTest, ReactiveConfigFromValidString) {
   scoped_ptr<BackgroundTracingConfigImpl> config;
 
   config = ReadFromJSONString(
       "{\"mode\":\"REACTIVE_TRACING_MODE\",\"configs\": [{\"rule\": "
       "\"TRACE_ON_NAVIGATION_UNTIL_TRIGGER_OR_FULL\", "
       "\"category\": \"BENCHMARK\", \"trigger_name\": \"foo\"}]}");
   EXPECT_TRUE(config);
@@ skipping 130 matching lines @@
     second_dict->SetInteger("trigger_delay", 10);
     config->AddPreemptiveRule(second_dict.get());
 
     EXPECT_EQ(ConfigToString(config.get()),
               "{\"category\":\"BENCHMARK\",\"configs\":[{\"histogram_lower_"
               "value\":1,\"histogram_name\":\"foo\",\"histogram_repeat\":true,"
               "\"histogram_upper_value\":2,\"rule\":\"MONITOR_AND_DUMP_WHEN_"
               "SPECIFIC_HISTOGRAM_AND_VALUE\",\"trigger_delay\":10}],\"mode\":"
               "\"PREEMPTIVE_TRACING_MODE\"}");
   }
-
-  {
-    config.reset(
-        new BackgroundTracingConfigImpl(BackgroundTracingConfig::PREEMPTIVE));
-    config->set_category_preset(BackgroundTracingConfigImpl::BENCHMARK_DEEP);
-
-    scoped_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
-    dict->SetString("rule", "MONITOR_AND_DUMP_WHEN_TRIGGER_NAMED");
-    dict->SetString("trigger_name", "foo1");
-    config->AddPreemptiveRule(dict.get());
-
-    config->scenario_name_ = "my_awesome_experiment";
-    config->enable_blink_features_ = "FasterWeb1,FasterWeb2";
-    config->disable_blink_features_ = "SlowerWeb1,SlowerWeb2";
-
-    EXPECT_EQ(ConfigToString(config.get()),
-              "{\"category\":\"BENCHMARK_DEEP\",\"configs\":[{\"rule\":"
-              "\"MONITOR_AND_DUMP_WHEN_TRIGGER_NAMED\",\"trigger_name\":"
-              "\"foo1\"}],\"disable_blink_features\":\"SlowerWeb1,SlowerWeb2\","
-              "\"enable_blink_features\":\"FasterWeb1,FasterWeb2\","
-              "\"mode\":\"PREEMPTIVE_TRACING_MODE\","
-              "\"scenario_name\":\"my_awesome_experiment\"}");
-  }
 }
 
 TEST_F(BackgroundTracingConfigTest, InvalidPreemptiveConfigToString) {
   scoped_ptr<BackgroundTracingConfigImpl> config;
 
   {
     config.reset(
         new BackgroundTracingConfigImpl(BackgroundTracingConfig::PREEMPTIVE));
 
     scoped_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
@@ skipping 79 matching lines @@
         ConfigToString(config.get()),
         "{\"configs\":[{\"category\":\"BENCHMARK_DEEP\",\"rule\":\"TRACE_"
         "ON_NAVIGATION_UNTIL_TRIGGER_OR_FULL\",\"trigger_name\":\"foo1\"},{"
         "\"category\":\"BENCHMARK_DEEP\",\"rule\":"
         "\"TRACE_ON_NAVIGATION_UNTIL_TRIGGER_OR_FULL\",\"trigger_name\":"
         "\"foo2\"}],\"mode\":\"REACTIVE_TRACING_MODE\"}");
   }
 }
 
 }  // namspace content