Remove stl_util use from domain_reliability.

components/domain_reliability/monitor.cc

 // Copyright 2014 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 "components/domain_reliability/monitor.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"

[davidben, 2016/08/30 00:52:54]

I think this is no longer needed?

[Avi (use Gerrit), 2016/08/30 16:11:26]

There's still a base::WrapUnique in the file on line 259...

 #include "base/profiler/scoped_tracker.h"
 #include "base/single_thread_task_runner.h"
 #include "base/task_runner.h"
 #include "components/domain_reliability/baked_in_configs.h"
 #include "components/domain_reliability/google_configs.h"
 #include "components/domain_reliability/header.h"
 #include "components/domain_reliability/quic_error_mapping.h"
 #include "net/base/ip_endpoint.h"
 #include "net/base/load_flags.h"
 #include "net/base/net_errors.h"
@@ skipping 142 matching lines @@
     std::unique_ptr<const DomainReliabilityConfig> config =
         DomainReliabilityConfig::FromJSON(json);
     if (!config) {
       DLOG(WARNING) << "Baked-in Domain Reliability config failed to parse: "
                     << json;
       continue;
     }
     context_manager_.AddContextForConfig(std::move(config));
   }
 
-  std::vector<DomainReliabilityConfig*> google_configs;
+  std::vector<std::unique_ptr<DomainReliabilityConfig>> google_configs;
   GetAllGoogleConfigs(&google_configs);
-  for (auto* google_config : google_configs)
-    context_manager_.AddContextForConfig(base::WrapUnique(google_config));
+  for (auto& google_config : google_configs)
+    context_manager_.AddContextForConfig(std::move(google_config));
 }
 
 void DomainReliabilityMonitor::SetDiscardUploads(bool discard_uploads) {
   DCHECK(OnNetworkThread());
   DCHECK(moved_to_network_thread_);
   DCHECK(uploader_);
 
   uploader_->set_discard_uploads(discard_uploads);
   discard_uploads_set_ = true;
 }
@@ skipping 61 matching lines @@
   return context_manager_.AddContextForConfig(std::move(config));
 }
 
 std::unique_ptr<DomainReliabilityContext>
 DomainReliabilityMonitor::CreateContextForConfig(
     std::unique_ptr<const DomainReliabilityConfig> config) {
   DCHECK(OnNetworkThread());
   DCHECK(config);
   DCHECK(config->IsValid());
 
   return base::WrapUnique(new DomainReliabilityContext(

[Avi (use Gerrit), 2016/08/30 16:11:25]

...here.

       time_.get(), scheduler_params_, upload_reporter_string_,
       &last_network_change_time_, &dispatcher_, uploader_.get(),
       std::move(config)));
 }
 
 DomainReliabilityMonitor::RequestInfo::RequestInfo() {}
 
 DomainReliabilityMonitor::RequestInfo::RequestInfo(
     const net::URLRequest& request)
     : url(request.url()),
@@ skipping 145 matching lines @@
       break;
   }
 }
 
 base::WeakPtr<DomainReliabilityMonitor>
 DomainReliabilityMonitor::MakeWeakPtr() {
   return weak_factory_.GetWeakPtr();
 }
 
 }  // namespace domain_reliability