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

Unified Diff: content/shell/common/shell_messages.h

Issue 1689283003: Remove duplication between TestRunner's fields and LayoutDumpFlags struct. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Despite LayoutDumpFlags constructor, we still need to set the flags in TestRunner::Reset. 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/shell/common/shell_messages.h
diff --git a/content/shell/common/shell_messages.h b/content/shell/common/shell_messages.h
index 684447408ffaeddec1740a40b1b75ae3c0a149ec..acbbfb019d2c9c4adee1bf88332a0a6700247b96 100644
--- a/content/shell/common/shell_messages.h
+++ b/content/shell/common/shell_messages.h
@@ -28,15 +28,13 @@ IPC_STRUCT_TRAITS_MEMBER(expected_pixel_hash)
IPC_STRUCT_TRAITS_MEMBER(initial_size)
IPC_STRUCT_TRAITS_END()
-IPC_ENUM_TRAITS_MIN_MAX_VALUE(
- test_runner::LayoutDumpMode,
- test_runner::LayoutDumpMode::DUMP_AS_TEXT,
- test_runner::LayoutDumpMode::DUMP_SCROLL_POSITIONS)
-
IPC_STRUCT_TRAITS_BEGIN(test_runner::LayoutDumpFlags)
- IPC_STRUCT_TRAITS_MEMBER(main_dump_mode)
- IPC_STRUCT_TRAITS_MEMBER(dump_as_printed)
- IPC_STRUCT_TRAITS_MEMBER(dump_child_frames)
+ IPC_STRUCT_TRAITS_MEMBER(dump_as_text)
+ IPC_STRUCT_TRAITS_MEMBER(dump_child_frames_as_text)
+ IPC_STRUCT_TRAITS_MEMBER(dump_as_markup)
+ IPC_STRUCT_TRAITS_MEMBER(dump_child_frames_as_markup)
+ IPC_STRUCT_TRAITS_MEMBER(dump_child_frame_scroll_positions)
+ IPC_STRUCT_TRAITS_MEMBER(is_printing)
IPC_STRUCT_TRAITS_MEMBER(dump_line_box_trees)
IPC_STRUCT_TRAITS_MEMBER(debug_render_tree)
IPC_STRUCT_TRAITS_END()
« no previous file with comments | « content/shell/browser/layout_test/blink_test_controller.cc ('k') | content/shell/renderer/layout_test/blink_test_runner.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698