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

Side by Side 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, 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/renderer/render_frame_impl.h" 5 #include "content/renderer/render_frame_impl.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 4434 matching lines...) Expand 10 before | Expand all | Expand 10 after
4445 // dependencies (crbug.com/392171 DownloadRequestHandle, SaveFileManager, 4445 // dependencies (crbug.com/392171 DownloadRequestHandle, SaveFileManager,
4446 // ResourceDispatcherHostImpl, MediaStreamUIProxy, 4446 // ResourceDispatcherHostImpl, MediaStreamUIProxy,
4447 // SpeechRecognitionDispatcherHost and possibly others). They look up the view 4447 // SpeechRecognitionDispatcherHost and possibly others). They look up the view
4448 // based on the ID stored in the resource requests. Once those dependencies 4448 // based on the ID stored in the resource requests. Once those dependencies
4449 // are unwound or moved to RenderFrameHost (crbug.com/304341) we can move the 4449 // are unwound or moved to RenderFrameHost (crbug.com/304341) we can move the
4450 // client to be based on the routing_id of the RenderFrameHost. 4450 // client to be based on the routing_id of the RenderFrameHost.
4451 params.render_view_routing_id = render_view_->routing_id(); 4451 params.render_view_routing_id = render_view_->routing_id();
4452 params.socket_address.set_host(response.remoteIPAddress().utf8()); 4452 params.socket_address.set_host(response.remoteIPAddress().utf8());
4453 params.socket_address.set_port(response.remotePort()); 4453 params.socket_address.set_port(response.remotePort());
4454 WebURLResponseExtraDataImpl* extra_data = GetExtraDataFromResponse(response); 4454 WebURLResponseExtraDataImpl* extra_data = GetExtraDataFromResponse(response);
4455 if (extra_data) 4455 if (extra_data) {
4456 params.was_fetched_via_proxy = extra_data->was_fetched_via_proxy(); 4456 params.was_fetched_via_proxy = extra_data->was_fetched_via_proxy();
4457 params.used_data_reduction_proxy = extra_data->used_data_reduction_proxy();
4458 params.is_using_lofi = extra_data->is_using_lofi();
4459 }
4457 params.was_within_same_page = navigation_state->WasWithinSamePage(); 4460 params.was_within_same_page = navigation_state->WasWithinSamePage();
4458 params.security_info = response.securityInfo(); 4461 params.security_info = response.securityInfo();
4459 4462
4460 // Set the origin of the frame. This will be replicated to the corresponding 4463 // Set the origin of the frame. This will be replicated to the corresponding
4461 // RenderFrameProxies in other processes. 4464 // RenderFrameProxies in other processes.
4462 // TODO(alexmos): Origins for URLs with non-standard schemes are excluded due 4465 // TODO(alexmos): Origins for URLs with non-standard schemes are excluded due
4463 // to https://crbug.com/439608 and will be replicated as unique origins. 4466 // to https://crbug.com/439608 and will be replicated as unique origins.
4464 if (!is_swapped_out_) { 4467 if (!is_swapped_out_) {
4465 std::string scheme = frame->document().securityOrigin().protocol().utf8(); 4468 std::string scheme = frame->document().securityOrigin().protocol().utf8();
4466 if (url::IsStandard(scheme.c_str(), 4469 if (url::IsStandard(scheme.c_str(),
(...skipping 1635 matching lines...) Expand 10 before | Expand all | Expand 10 after
6102 int match_count, 6105 int match_count,
6103 int ordinal, 6106 int ordinal,
6104 const WebRect& selection_rect, 6107 const WebRect& selection_rect,
6105 bool final_status_update) { 6108 bool final_status_update) {
6106 Send(new FrameHostMsg_Find_Reply(routing_id_, request_id, match_count, 6109 Send(new FrameHostMsg_Find_Reply(routing_id_, request_id, match_count,
6107 selection_rect, ordinal, 6110 selection_rect, ordinal,
6108 final_status_update)); 6111 final_status_update));
6109 } 6112 }
6110 6113
6111 } // namespace content 6114 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698