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

Unified Diff: components/test_runner/test_runner.h

Issue 1715573002: Replicating LayoutDumpFlags across OOPIFs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@replicating-pixel-dump-flag
Patch Set: Removed unneeded include. 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
« no previous file with comments | « components/test_runner/layout_dump_flags.cc ('k') | components/test_runner/test_runner.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/test_runner/test_runner.h
diff --git a/components/test_runner/test_runner.h b/components/test_runner/test_runner.h
index fe42c0a36d5bb661c5b37d2ef1d2c59f8cd5e896..d6d5c24fe9e113fcf3e1dae0eeddb0c5fd33491e 100644
--- a/components/test_runner/test_runner.h
+++ b/components/test_runner/test_runner.h
@@ -74,6 +74,8 @@ class TestRunner : public WebTestRunner,
bool ShouldDumpAsAudio() const override;
void GetAudioData(std::vector<unsigned char>* buffer_view) const override;
const LayoutDumpFlags& GetLayoutDumpFlags() override;
+ void ReplicateLayoutDumpFlagsChanges(
+ const base::DictionaryValue& changed_values) override;
bool HasCustomTextDump(std::string* custom_text_dump) const override;
bool ShouldDumpBackForwardList() const override;
blink::WebContentSettingsClient* GetWebContentSettings() const override;
@@ -622,6 +624,9 @@ class TestRunner : public WebTestRunner,
void GetManifestThen(v8::Local<v8::Function> callback);
+ // Takes care of notifying the delegate after a change to layout dump flags.
+ void OnLayoutDumpFlagsChanged();
+
///////////////////////////////////////////////////////////////////////////
// Internal helpers
@@ -661,9 +666,6 @@ class TestRunner : public WebTestRunner,
// setCloseRemainingWindowsWhenComplete().
bool close_remaining_windows_;
- // If true, don't dump output until notifyDone is called.
- bool wait_until_done_;
-
// If true, ends the test when a URL is loaded externally via
// WebFrameClient::loadURLExternally().
bool wait_until_external_url_load_;
@@ -704,9 +706,6 @@ class TestRunner : public WebTestRunner,
// command.
bool dump_editting_callbacks_;
- // If true, the test_shell will generate pixel results in DumpAsText mode
- bool generate_pixel_results_;
-
// Flags controlling what content gets dumped as a layout text result.
LayoutDumpFlags layout_dump_flags_;
« no previous file with comments | « components/test_runner/layout_dump_flags.cc ('k') | components/test_runner/test_runner.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698