Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(151)

Side by Side Diff: components/domain_reliability/monitor.cc

Issue 2257793002: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/domain_reliability/monitor.h" 5 #include "components/domain_reliability/monitor.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 238 matching lines...) Expand 10 before | Expand all | Expand 10 after
249 return context_manager_.AddContextForConfig(std::move(config)); 249 return context_manager_.AddContextForConfig(std::move(config));
250 } 250 }
251 251
252 std::unique_ptr<DomainReliabilityContext> 252 std::unique_ptr<DomainReliabilityContext>
253 DomainReliabilityMonitor::CreateContextForConfig( 253 DomainReliabilityMonitor::CreateContextForConfig(
254 std::unique_ptr<const DomainReliabilityConfig> config) { 254 std::unique_ptr<const DomainReliabilityConfig> config) {
255 DCHECK(OnNetworkThread()); 255 DCHECK(OnNetworkThread());
256 DCHECK(config); 256 DCHECK(config);
257 DCHECK(config->IsValid()); 257 DCHECK(config->IsValid());
258 258
259 return base::WrapUnique(new DomainReliabilityContext( 259 return base::MakeUnique<DomainReliabilityContext>(
260 time_.get(), scheduler_params_, upload_reporter_string_, 260 time_.get(), scheduler_params_, upload_reporter_string_,
261 &last_network_change_time_, &dispatcher_, uploader_.get(), 261 &last_network_change_time_, &dispatcher_, uploader_.get(),
262 std::move(config))); 262 std::move(config));
263 } 263 }
264 264
265 DomainReliabilityMonitor::RequestInfo::RequestInfo() {} 265 DomainReliabilityMonitor::RequestInfo::RequestInfo() {}
266 266
267 DomainReliabilityMonitor::RequestInfo::RequestInfo( 267 DomainReliabilityMonitor::RequestInfo::RequestInfo(
268 const net::URLRequest& request) 268 const net::URLRequest& request)
269 : url(request.url()), 269 : url(request.url()),
270 status(request.status()), 270 status(request.status()),
271 response_info(request.response_info()), 271 response_info(request.response_info()),
272 load_flags(request.load_flags()), 272 load_flags(request.load_flags()),
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
415 break; 415 break;
416 } 416 }
417 } 417 }
418 418
419 base::WeakPtr<DomainReliabilityMonitor> 419 base::WeakPtr<DomainReliabilityMonitor>
420 DomainReliabilityMonitor::MakeWeakPtr() { 420 DomainReliabilityMonitor::MakeWeakPtr() {
421 return weak_factory_.GetWeakPtr(); 421 return weak_factory_.GetWeakPtr();
422 } 422 }
423 423
424 } // namespace domain_reliability 424 } // namespace domain_reliability
OLDNEW
« no previous file with comments | « components/dom_distiller/core/dom_distiller_store_unittest.cc ('k') | components/drive/chromeos/change_list_loader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698