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

Unified Diff: net/url_request/url_fetcher_impl.cc

Issue 2846873002: Network traffic annotation added to URLFetcher. (Closed)
Patch Set: Comment addressed. Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: net/url_request/url_fetcher_impl.cc
diff --git a/net/url_request/url_fetcher_impl.cc b/net/url_request/url_fetcher_impl.cc
index bdeee40be40d42ab001112c887eb4e6cfad7ffa8..0b180ac3478fdab438971bd8bffe8b55f6305ffc 100644
--- a/net/url_request/url_fetcher_impl.cc
+++ b/net/url_request/url_fetcher_impl.cc
@@ -17,11 +17,13 @@ namespace net {
static URLFetcherFactory* g_factory = NULL;
-URLFetcherImpl::URLFetcherImpl(const GURL& url,
- RequestType request_type,
- URLFetcherDelegate* d)
- : core_(new URLFetcherCore(this, url, request_type, d)) {
-}
+URLFetcherImpl::URLFetcherImpl(
+ const GURL& url,
+ RequestType request_type,
+ URLFetcherDelegate* d,
+ const net::NetworkTrafficAnnotationTag traffic_annotation)
asanka 2017/05/03 14:45:44 const unnecessary for pass-by-value (here and else
Ramin Halavati 2017/05/03 14:58:50 Done.
+ : core_(
+ new URLFetcherCore(this, url, request_type, d, traffic_annotation)) {}
URLFetcherImpl::~URLFetcherImpl() {
core_->Stop();

Powered by Google App Engine
This is Rietveld 408576698