| commit | 718fd6c00a69d8b7a29959a8b5ea2b6c208fd78c | [log] [tgz] |
|---|---|---|
| author | Jay Civelli <[email protected]> | Fri May 11 21:04:27 2018 |
| committer | Commit Bot <[email protected]> | Fri May 11 21:04:27 2018 |
| tree | 9583816a0953abd4919825e916f5e434b59b7879 | |
| parent | 8e75252451a3a2aff0183228890344180ccd3aa1 [diff] [blame] |
Add more info to ResourceLoadInfo Adds additional information to the WebContentsObserver ResourceLoadComplete notification that is required by the page load metrics code: - host IP and port for the request - the total bytes count - the raw bytes count Also passing along the RenderFrameHost as part of the callback. Change-Id: I9646221498312800488660d6b1b76abed467031b Reviewed-on: https://chromium-review.googlesource.com/1050927 Reviewed-by: Tom Sepez <[email protected]> Reviewed-by: John Abd-El-Malek <[email protected]> Commit-Queue: Jay Civelli <[email protected]> Cr-Commit-Position: refs/heads/master@{#558016}
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc index 0bb17c9..464b2f90 100644 --- a/content/browser/web_contents/web_contents_impl.cc +++ b/content/browser/web_contents/web_contents_impl.cc
@@ -4154,9 +4154,8 @@ void WebContentsImpl::ResourceLoadComplete( RenderFrameHost* render_frame_host, mojom::ResourceLoadInfoPtr resource_load_info) { - bool is_main_frame = !render_frame_host->GetParent(); for (auto& observer : observers_) { - observer.ResourceLoadComplete(*resource_load_info, is_main_frame); + observer.ResourceLoadComplete(render_frame_host, *resource_load_info); } }