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

Unified Diff: content/browser/loader/resource_message_filter.cc

Issue 2893233002: Network traffic annotation added to URLLoaderImpl. (Closed)
Patch Set: Changed ParamTraits to StructTraits. Created 3 years, 6 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: content/browser/loader/resource_message_filter.cc
diff --git a/content/browser/loader/resource_message_filter.cc b/content/browser/loader/resource_message_filter.cc
index 57d7409ec26fae0eb45c425e85ae5e89d81122c0..2c0c47a0d10162483d2d83c7a7e38440f36899e4 100644
--- a/content/browser/loader/resource_message_filter.cc
+++ b/content/browser/loader/resource_message_filter.cc
@@ -90,11 +90,13 @@ void ResourceMessageFilter::CreateLoaderAndStart(
int32_t request_id,
uint32_t options,
const ResourceRequest& url_request,
- mojom::URLLoaderClientPtr client) {
+ mojom::URLLoaderClientPtr client,
+ const net::MutableNetworkTrafficAnnotationTag& traffic_annotation) {
DCHECK_EQ(options, mojom::kURLLoadOptionNone);
URLLoaderFactoryImpl::CreateLoaderAndStart(
requester_info_.get(), std::move(request), routing_id, request_id,
- url_request, std::move(client));
+ url_request, std::move(client),
+ static_cast<net::NetworkTrafficAnnotationTag>(traffic_annotation));
jochen (gone - plz use gerrit) 2017/06/08 17:00:58 just wondering, why doesn't this have to be const
Ramin Halavati 2017/06/09 07:08:53 Done.
}
void ResourceMessageFilter::SyncLoad(int32_t routing_id,

Powered by Google App Engine
This is Rietveld 408576698