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

Unified Diff: components/test_runner/web_test_runner.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: Rebasing... 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: components/test_runner/web_test_runner.h
diff --git a/components/test_runner/web_test_runner.h b/components/test_runner/web_test_runner.h
index b5040dff2f7cf37b2163c5aed262650fd9e22e19..e6fbd08d844a6a4cfe0e4ff5f8818ec210f34a20 100644
--- a/components/test_runner/web_test_runner.h
+++ b/components/test_runner/web_test_runner.h
@@ -8,7 +8,9 @@
#include <string>
#include <vector>
-#include "components/test_runner/layout_dump_flags.h"
+namespace test_runner {
+struct LayoutDumpFlags;
+}
namespace blink {
class WebContentSettingsClient;
@@ -33,7 +35,7 @@ class WebTestRunner {
// Gets layout dump flags (i.e. dump-as-text or dump-as-markup) requested
// by the test (i.e. via testRunner.dumpAsText() called from javascript).
- virtual LayoutDumpFlags GetLayoutDumpFlags() = 0;
+ virtual const LayoutDumpFlags& GetLayoutDumpFlags() = 0;
// If custom text dump is present (i.e. if testRunner.setCustomTextOutput has
// been called from javascript), then returns |true| and populates the

Powered by Google App Engine
This is Rietveld 408576698