Domain Reliability: Don't crash on shutdown with uploads pending

chrome/browser/net/chrome_network_delegate.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_
 #define CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_
 
 #include <stdint.h>
 
 #include <memory>
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "base/files/file_path.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/values.h"
 #include "build/build_config.h"
 #include "components/data_use_measurement/content/data_use_measurement.h"
+#include "components/domain_reliability/monitor.h"
 #include "components/metrics/data_use_tracker.h"
 #include "components/prefs/pref_member.h"
 #include "net/base/network_delegate_impl.h"
 
 class ChromeExtensionsNetworkDelegate;
 class PrefService;
 
 template<class T> class PrefMember;
 
 typedef PrefMember<bool> BooleanPrefMember;
@@ skipping 79 matching lines @@
       BooleanPrefMember* force_youtube_safety_mode) {
     force_youtube_safety_mode_ = force_youtube_safety_mode;
   }
 
   void set_allowed_domains_for_apps(
       StringPrefMember* allowed_domains_for_apps) {
     allowed_domains_for_apps_ = allowed_domains_for_apps;
   }
 
   void set_domain_reliability_monitor(
-      domain_reliability::DomainReliabilityMonitor* monitor) {
-    domain_reliability_monitor_ = monitor;
+      std::unique_ptr<domain_reliability::DomainReliabilityMonitor> monitor) {
+    domain_reliability_monitor_ = std::move(monitor);
   }
 
   void set_data_use_aggregator(
       data_usage::DataUseAggregator* data_use_aggregator,
       bool is_data_usage_off_the_record);
 
   // Binds the pref members to |pref_service| and moves them to the IO thread.
   // |enable_referrers| cannot be NULL, the others can.
   // This method should be called on the UI thread.
   static void InitializePrefsOnUIThread(
@@ skipping 70 matching lines @@
 
   // Weak, owned by our owner.
   BooleanPrefMember* enable_referrers_;
   BooleanPrefMember* enable_do_not_track_;
   BooleanPrefMember* force_google_safe_search_;
   BooleanPrefMember* force_youtube_safety_mode_;
   StringPrefMember* allowed_domains_for_apps_;
 
   // Weak, owned by our owner.
   const policy::URLBlacklistManager* url_blacklist_manager_;
-  domain_reliability::DomainReliabilityMonitor* domain_reliability_monitor_;
+  std::unique_ptr<domain_reliability::DomainReliabilityMonitor>
+      domain_reliability_monitor_;
 
   // When true, allow access to all file:// URLs.
   static bool g_allow_file_access_;
 
   // Component to measure data use.
   data_use_measurement::DataUseMeasurement data_use_measurement_;
 
   bool experimental_web_platform_features_enabled_;
 
   // Aggregates and reports network usage.
   data_usage::DataUseAggregator* data_use_aggregator_;
   // Controls whether network usage is reported as being off the record.
   bool is_data_usage_off_the_record_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeNetworkDelegate);
 };
 
 #endif  // CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_