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

Unified Diff: content/test/test_render_frame_host.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/test/test_render_frame_host.cc
diff --git a/content/test/test_render_frame_host.cc b/content/test/test_render_frame_host.cc
index f158dbcc40fb58a18191d134faa0b39786d2807d..74baa3663a820016f80356d17792763e2fd5f4df 100644
--- a/content/test/test_render_frame_host.cc
+++ b/content/test/test_render_frame_host.cc
@@ -53,21 +53,22 @@ TestRenderFrameHost::TestRenderFrameHost(SiteInstance* site_instance,
render_view_host,
delegate,
rwh_delegate,
frame_tree,
frame_tree_node,
routing_id,
widget_routing_id,
flags),
child_creation_observer_(delegate ? delegate->GetAsWebContents() : NULL),
contents_mime_type_("text/html"),
- simulate_history_list_was_cleared_(false) {}
+ simulate_history_list_was_cleared_(false),
+ simulate_fetch_via_data_reduction_proxy_(false) {}
TestRenderFrameHost::~TestRenderFrameHost() {
}
TestRenderViewHost* TestRenderFrameHost::GetRenderViewHost() {
return static_cast<TestRenderViewHost*>(
RenderFrameHostImpl::GetRenderViewHost());
}
MockRenderProcessHost* TestRenderFrameHost::GetProcess() {
@@ -285,20 +286,21 @@ void TestRenderFrameHost::SendNavigateWithParameters(
params.should_update_history = true;
params.did_create_new_entry = did_create_new_entry;
params.gesture = NavigationGestureUser;
params.contents_mime_type = contents_mime_type_;
params.is_post = false;
params.http_status_code = response_code;
params.socket_address.set_host("2001:db8::1");
params.socket_address.set_port(80);
params.history_list_was_cleared = simulate_history_list_was_cleared_;
params.original_request_url = url_copy;
+ params.used_data_reduction_proxy = simulate_fetch_via_data_reduction_proxy_;
// In most cases, the origin will match the URL's origin. Tests that need to
// check corner cases (like about:blank) should specify the origin param
// manually.
url::Origin origin(url_copy);
params.origin = origin;
url::Replacements<char> replacements;
replacements.ClearRef();
params.was_within_same_page =

Powered by Google App Engine
This is Rietveld 408576698