Remove --reduce-security-for-stats-collection-tests switch

This change is per jam@'s review comment on https://codereview.chromium.org/12389073/ . The switch is redundant with --enable-stats-collection-bindings .

BUG=None
TBR=tonyg

Review URL: https://chromiumcodereview.appspot.com/15804006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@203122 0039d316-1c4b-4281-b951-d872f2087c98

content/browser/histogram_message_filter.cc

@@ -52,15 +52,9 @@ void HistogramMessageFilter::OnGetBrowserHistogram(
   bool using_stats_collection_controller =
       CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kStatsCollectionController);
-  bool reduced_security =
-      CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kReduceSecurityForStatsCollectionTests);
-
-  if (!using_stats_collection_controller || !reduced_security) {
+  if (!using_stats_collection_controller) {
     LOG(ERROR) << "Attempt at reading browser histogram without specifying "
-               << "--" << switches::kStatsCollectionController << " and "
-               << "--" << switches::kReduceSecurityForStatsCollectionTests
-               << " switches.";
+               << "--" << switches::kStatsCollectionController << " switch.";
     return;
   }
   base::HistogramBase* histogram =

content/public/common/content_switches.cc

@@ -245,11 +245,6 @@ const char kDisableXSSAuditor[]             = "disable-xss-auditor";
 // based tests.
 const char kDomAutomationController[]       = "dom-automation";
 
-// Loosen security. Needed for tests using some of the functionality of
-// |DOMAutomationController|.
-const char kReduceSecurityForStatsCollectionTests[] =
-    "reduce-security-for-stats-collection-tests";
-
 // Specifies if the |StatsCollectionController| needs to be bound in the
 // renderer. This binding happens on per-frame basis and hence can potentially
 // be a performance bottleneck. One should only enable it when running a test

content/public/common/content_switches.h

@@ -92,7 +92,6 @@ extern const char kDisableWebSecurity[];
 extern const char kEnableExperimentalWebSocket[];
 extern const char kDisableXSSAuditor[];
 CONTENT_EXPORT extern const char kDomAutomationController[];
-CONTENT_EXPORT extern const char kReduceSecurityForStatsCollectionTests[];
 CONTENT_EXPORT extern const char kStatsCollectionController[];
 CONTENT_EXPORT extern const char kEnableAcceleratedFilters[];
 extern const char kEnableAccessibilityLogging[];

tools/perf/perf_tools/memory_measurement.py

@@ -38,8 +38,6 @@ class MemoryMeasurement(page_measurement.PageMeasurement):
     # pages.
     options.AppendExtraBrowserArg('--disable-histogram-customizer')
     options.AppendExtraBrowserArg('--memory-metrics')
-    options.AppendExtraBrowserArg(
-        '--reduce-security-for-stats-collection-tests')
 
     # Old commandline flags used for reference builds.
     options.AppendExtraBrowserArg('--dom-automation')

tools/perf/perf_tools/startup_measurement.py

@@ -20,8 +20,6 @@ class PerfWarm(page_measurement.PageMeasurement):
 
   def CustomizeBrowserOptions(self, options):
     options.AppendExtraBrowserArg('--enable-stats-collection-bindings')
-    options.AppendExtraBrowserArg(
-          '--reduce-security-for-stats-collection-tests')
 
     # Old commandline flags used for reference builds.
     options.AppendExtraBrowserArg('--dom-automation')