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

Side by Side 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, 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 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 "content/child/web_url_loader_impl.h" 5 #include "content/child/web_url_loader_impl.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <memory> 10 #include <memory>
(...skipping 584 matching lines...) Expand 10 before | Expand all | Expand 10 after
595 resource_request->enable_upload_progress = request.reportUploadProgress(); 595 resource_request->enable_upload_progress = request.reportUploadProgress();
596 if (request.getRequestContext() == 596 if (request.getRequestContext() ==
597 WebURLRequest::RequestContextXMLHttpRequest && 597 WebURLRequest::RequestContextXMLHttpRequest &&
598 (url.has_username() || url.has_password())) { 598 (url.has_username() || url.has_password())) {
599 resource_request->do_not_prompt_for_login = true; 599 resource_request->do_not_prompt_for_login = true;
600 } 600 }
601 resource_request->report_raw_headers = request.reportRawHeaders(); 601 resource_request->report_raw_headers = request.reportRawHeaders();
602 resource_request->previews_state = 602 resource_request->previews_state =
603 static_cast<PreviewsState>(request.getPreviewsState()); 603 static_cast<PreviewsState>(request.getPreviewsState());
604 604
605 resource_request->devtools_agent_host_id =
606 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.
607 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.
608
605 // PlzNavigate: during navigation, the renderer should request a stream which 609 // PlzNavigate: during navigation, the renderer should request a stream which
606 // contains the body of the response. The network request has already been 610 // contains the body of the response. The network request has already been
607 // made by the browser. 611 // made by the browser.
608 if (stream_override_.get()) { 612 if (stream_override_.get()) {
609 CHECK(IsBrowserSideNavigationEnabled()); 613 CHECK(IsBrowserSideNavigationEnabled());
610 DCHECK(!sync_load_response); 614 DCHECK(!sync_load_response);
611 DCHECK_NE(WebURLRequest::FrameTypeNone, request.getFrameType()); 615 DCHECK_NE(WebURLRequest::FrameTypeNone, request.getFrameType());
612 resource_request->resource_body_stream_url = stream_override_->stream_url; 616 resource_request->resource_body_stream_url = stream_override_->stream_url;
613 } 617 }
614 618
(...skipping 634 matching lines...) Expand 10 before | Expand all | Expand 10 after
1249 int intra_priority_value) { 1253 int intra_priority_value) {
1250 context_->DidChangePriority(new_priority, intra_priority_value); 1254 context_->DidChangePriority(new_priority, intra_priority_value);
1251 } 1255 }
1252 1256
1253 void WebURLLoaderImpl::setLoadingTaskRunner( 1257 void WebURLLoaderImpl::setLoadingTaskRunner(
1254 base::SingleThreadTaskRunner* loading_task_runner) { 1258 base::SingleThreadTaskRunner* loading_task_runner) {
1255 context_->SetTaskRunner(loading_task_runner); 1259 context_->SetTaskRunner(loading_task_runner);
1256 } 1260 }
1257 1261
1258 } // namespace content 1262 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698