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

Unified Diff: webkit/tools/test_shell/layout_test_controller.cc

Issue 21192: Add support for the "repaint" layout tests (LayoutTests/fast/repaint/*), whic... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 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: webkit/tools/test_shell/layout_test_controller.cc
===================================================================
--- webkit/tools/test_shell/layout_test_controller.cc (revision 9433)
+++ webkit/tools/test_shell/layout_test_controller.cc (working copy)
@@ -189,7 +189,7 @@
}
void LayoutTestController::dumpAsText(const CppArgumentList& args,
- CppVariant* result) {
+ CppVariant* result) {
dump_as_text_ = true;
result->SetNull();
}
@@ -279,7 +279,7 @@
class WorkItemBackForward : public LayoutTestController::WorkItem {
public:
- WorkItemBackForward(int distance) : distance_(distance) {}
+ explicit WorkItemBackForward(int distance) : distance_(distance) {}
void Run(TestShell* shell) {
shell->GoBackOrForward(distance_);
}
@@ -316,7 +316,7 @@
class WorkItemScript : public LayoutTestController::WorkItem {
public:
- WorkItemScript(const string& script) : script_(script) {}
+ explicit WorkItemScript(const string& script) : script_(script) {}
void Run(TestShell* shell) {
wstring url = L"javascript:" + UTF8ToWide(script_);
shell->LoadURL(url.c_str());
@@ -663,6 +663,7 @@
void LayoutTestController::display(
const CppArgumentList& args, CppVariant* result) {
+ shell_->webViewHost()->DisplayForRepaint();
result->SetNull();
}

Powered by Google App Engine
This is Rietveld 408576698