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

Unified Diff: content/shell/renderer/layout_test/layout_test_render_frame_observer.cc

Issue 1715573002: Replicating LayoutDumpFlags across OOPIFs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@replicating-pixel-dump-flag
Patch Set: Rebasing... 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 side-by-side diff with in-line comments
Download patch
Index: content/shell/renderer/layout_test/layout_test_render_frame_observer.cc
diff --git a/content/shell/renderer/layout_test/layout_test_render_frame_observer.cc b/content/shell/renderer/layout_test/layout_test_render_frame_observer.cc
index 7376b367b9d2a305dd5f65a87c21628cb9693798..a17c9fcad26736d67e19a3eec82b800e1b53e00f 100644
--- a/content/shell/renderer/layout_test/layout_test_render_frame_observer.cc
+++ b/content/shell/renderer/layout_test/layout_test_render_frame_observer.cc
@@ -34,6 +34,8 @@ bool LayoutTestRenderFrameObserver::OnMessageReceived(
bool handled = true;
IPC_BEGIN_MESSAGE_MAP(LayoutTestRenderFrameObserver, message)
IPC_MESSAGE_HANDLER(ShellViewMsg_LayoutDumpRequest, OnLayoutDumpRequest)
+ IPC_MESSAGE_HANDLER(ShellViewMsg_ReplicateLayoutDumpFlagsChanges,
+ OnReplicateLayoutDumpFlagsChanges)
IPC_MESSAGE_HANDLER(ShellViewMsg_ReplicateTestConfiguration,
OnReplicateTestConfiguration)
IPC_MESSAGE_HANDLER(ShellViewMsg_SetTestConfiguration,
@@ -44,18 +46,33 @@ bool LayoutTestRenderFrameObserver::OnMessageReceived(
return handled;
}
-void LayoutTestRenderFrameObserver::OnLayoutDumpRequest(
- const test_runner::LayoutDumpFlags& layout_dump_flags) {
+void LayoutTestRenderFrameObserver::OnLayoutDumpRequest() {
+ const test_runner::LayoutDumpFlags& layout_dump_flags =
+ LayoutTestRenderProcessObserver::GetInstance()
+ ->test_interfaces()
+ ->TestRunner()
+ ->GetLayoutDumpFlags();
std::string dump =
test_runner::DumpLayout(render_frame()->GetWebFrame(), layout_dump_flags);
Send(new ShellViewHostMsg_LayoutDumpResponse(routing_id(), dump));
}
+void LayoutTestRenderFrameObserver::OnReplicateLayoutDumpFlagsChanges(
+ const base::DictionaryValue& changed_layout_dump_flags) {
+ LayoutTestRenderProcessObserver::GetInstance()
+ ->test_interfaces()
+ ->TestRunner()
+ ->ReplicateLayoutDumpFlagsChanges(changed_layout_dump_flags);
+}
+
void LayoutTestRenderFrameObserver::OnReplicateTestConfiguration(
- const ShellTestConfiguration& test_config) {
+ const ShellTestConfiguration& test_config,
+ const base::DictionaryValue& accumulated_layout_dump_flags_changes) {
LayoutTestRenderProcessObserver::GetInstance()
->main_test_runner()
->OnReplicateTestConfiguration(test_config);
+
+ OnReplicateLayoutDumpFlagsChanges(accumulated_layout_dump_flags_changes);
}
void LayoutTestRenderFrameObserver::OnSetTestConfiguration(

Powered by Google App Engine
This is Rietveld 408576698