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

Unified Diff: content/shell/test_runner/web_test_runner.h

Issue 2963593002: Split DumpPixelsAsync in pixel_dump.h into more granular functions. (Closed)
Patch Set: Addressed CR feedback from alexmos@. Created 3 years, 5 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 | « content/shell/test_runner/test_runner_for_specific_view.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/shell/test_runner/web_test_runner.h
diff --git a/content/shell/test_runner/web_test_runner.h b/content/shell/test_runner/web_test_runner.h
index 5dd449c43aabd740ac72b77de7bd7e5ce90096b7..6efaec303381fc119100769d58f5a32460f9d614 100644
--- a/content/shell/test_runner/web_test_runner.h
+++ b/content/shell/test_runner/web_test_runner.h
@@ -56,8 +56,8 @@ class WebTestRunner {
// and calls |callback| with the result. Caller needs to ensure that
// |web_view| stays alive until |callback| is called.
virtual void DumpPixelsAsync(
- blink::WebView* web_view,
- const base::Callback<void(const SkBitmap&)>& callback) = 0;
+ blink::WebLocalFrame* frame,
+ base::OnceCallback<void(const SkBitmap&)> callback) = 0;
// Replicates changes to layout test runtime flags
// (i.e. changes that happened in another renderer).
« no previous file with comments | « content/shell/test_runner/test_runner_for_specific_view.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698