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

Unified Diff: content/renderer/render_frame_impl.cc

Issue 1721813002: Adding DRP specfic UMA for FirstContentfulPaint (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created a DRP PageLoadMetricsObserver Created 4 years, 10 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/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index ec975b5e8bd90cff456118229e8b216f512d2e6c..1bf8cd7177d445e844b23a5819c58d1d8bdf3a86 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -4445,22 +4445,25 @@ void RenderFrameImpl::SendDidCommitProvisionalLoad(
// dependencies (crbug.com/392171 DownloadRequestHandle, SaveFileManager,
// ResourceDispatcherHostImpl, MediaStreamUIProxy,
// SpeechRecognitionDispatcherHost and possibly others). They look up the view
// based on the ID stored in the resource requests. Once those dependencies
// are unwound or moved to RenderFrameHost (crbug.com/304341) we can move the
// client to be based on the routing_id of the RenderFrameHost.
params.render_view_routing_id = render_view_->routing_id();
params.socket_address.set_host(response.remoteIPAddress().utf8());
params.socket_address.set_port(response.remotePort());
WebURLResponseExtraDataImpl* extra_data = GetExtraDataFromResponse(response);
- if (extra_data)
+ if (extra_data) {
params.was_fetched_via_proxy = extra_data->was_fetched_via_proxy();
+ params.used_data_reduction_proxy = extra_data->used_data_reduction_proxy();
+ params.is_using_lofi = extra_data->is_using_lofi();
+ }
params.was_within_same_page = navigation_state->WasWithinSamePage();
params.security_info = response.securityInfo();
// Set the origin of the frame. This will be replicated to the corresponding
// RenderFrameProxies in other processes.
// TODO(alexmos): Origins for URLs with non-standard schemes are excluded due
// to https://crbug.com/439608 and will be replicated as unique origins.
if (!is_swapped_out_) {
std::string scheme = frame->document().securityOrigin().protocol().utf8();
if (url::IsStandard(scheme.c_str(),

Powered by Google App Engine
This is Rietveld 408576698