Domain Reliability: Don't create Service if metrics reporting disabled

chrome/browser/domain_reliability/service_factory.cc

Index: chrome/browser/domain_reliability/service_factory.cc
diff --git a/chrome/browser/domain_reliability/service_factory.cc b/chrome/browser/domain_reliability/service_factory.cc
index 0809074b7def79e2dd450288d97484ca1b6aa194..87ccf93e07b565cf75cbd84ec065af370d1857d8 100644
--- a/chrome/browser/domain_reliability/service_factory.cc
+++ b/chrome/browser/domain_reliability/service_factory.cc
@@ -6,7 +6,10 @@
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
+#include "base/prefs/pref_service.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/common/chrome_switches.h"
+#include "chrome/common/pref_names.h"
 #include "components/domain_reliability/service.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "content/public/browser/browser_context.h"
@@ -47,7 +50,10 @@ DomainReliabilityServiceFactory::GetInstance() {
 DomainReliabilityServiceFactory::DomainReliabilityServiceFactory()
     : BrowserContextKeyedServiceFactory(
           "DomainReliabilityService",
-          BrowserContextDependencyManager::GetInstance()) {}
+          BrowserContextDependencyManager::GetInstance()) {
+  reporting_pref_.Init(prefs::kMetricsReportingEnabled,
+                       g_browser_process->local_state());

[mmenke, 2014/09/02 18:45:40]

Do we even need this member variable?  Seems like we can just grab it in
BuildServiceInstanceFor.

[Deprecated (see juliatuttle), 2014/09/02 20:41:27]

Yeah, maybe you're right.

+}
 
 DomainReliabilityServiceFactory::~DomainReliabilityServiceFactory() {}
 
@@ -56,6 +62,9 @@ KeyedService* DomainReliabilityServiceFactory::BuildServiceInstanceFor(
   if (!IsDomainReliabilityMonitoringEnabled())
     return NULL;
 
+  if (!*reporting_pref_)
+    return NULL;
+
   return DomainReliabilityService::Create(
       kDomainReliabilityUploadReporterString);
 }