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

Unified Diff: content/child/web_url_loader_impl.cc

Issue 2776523005: Plumbing devtools agent host id and request id between processes (Closed)
Patch Set: Full plumbing Created 3 years, 9 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/child/web_url_loader_impl.cc
diff --git a/content/child/web_url_loader_impl.cc b/content/child/web_url_loader_impl.cc
index 3b5126aea04bd0e021f38aba6e4f2318b8745f87..9cce04d4c820fb6cb0b65f8287b3ec8541042013 100644
--- a/content/child/web_url_loader_impl.cc
+++ b/content/child/web_url_loader_impl.cc
@@ -602,6 +602,10 @@ void WebURLLoaderImpl::Context::Start(const WebURLRequest& request,
resource_request->previews_state =
static_cast<PreviewsState>(request.getPreviewsState());
+ resource_request->devtools_agent_host_id =
+ request.devToolsAgentHostId().latin1();
Sami 2017/03/29 16:41:42 utf8() instead of latin1()?
alex clarke (OOO till 29th) 2017/03/30 16:56:55 Done.
+ resource_request->devtools_request_id = request.devToolsRequestId().latin1();
Sami 2017/03/29 16:41:42 Ditto.
alex clarke (OOO till 29th) 2017/03/30 16:56:55 Done.
+
// PlzNavigate: during navigation, the renderer should request a stream which
// contains the body of the response. The network request has already been
// made by the browser.

Powered by Google App Engine
This is Rietveld 408576698