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

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

Issue 164225: Switch to WebFrame from the WebKit API.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 4 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 | « webkit/glue/webworker_impl.cc ('k') | webkit/tools/test_shell/plugin_tests.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/tools/test_shell/layout_test_controller.cc
===================================================================
--- webkit/tools/test_shell/layout_test_controller.cc (revision 22886)
+++ webkit/tools/test_shell/layout_test_controller.cc (working copy)
@@ -14,9 +14,9 @@
#include "base/message_loop.h"
#include "base/path_service.h"
#include "base/string_util.h"
+#include "webkit/api/public/WebFrame.h"
#include "webkit/api/public/WebScriptSource.h"
#include "webkit/glue/dom_operations.h"
-#include "webkit/glue/webframe.h"
#include "webkit/glue/webpreferences.h"
#include "webkit/glue/webview.h"
#include "webkit/tools/test_shell/test_navigation_controller.h"
@@ -329,7 +329,7 @@
public:
WorkItemLoadingScript(const string& script) : script_(script) {}
bool Run(TestShell* shell) {
- shell->webView()->GetMainFrame()->ExecuteScript(
+ shell->webView()->GetMainFrame()->executeScript(
WebScriptSource(WebString::fromUTF8(script_)));
return true; // TODO(darin): Did it really start a navigation?
}
@@ -341,7 +341,7 @@
public:
WorkItemNonLoadingScript(const string& script) : script_(script) {}
bool Run(TestShell* shell) {
- shell->webView()->GetMainFrame()->ExecuteScript(
+ shell->webView()->GetMainFrame()->executeScript(
WebScriptSource(WebString::fromUTF8(script_)));
return false;
}
@@ -380,7 +380,7 @@
void LayoutTestController::queueLoad(
const CppArgumentList& args, CppVariant* result) {
if (args.size() > 0 && args[0].isString()) {
- GURL current_url = shell_->webView()->GetMainFrame()->GetURL();
+ GURL current_url = shell_->webView()->GetMainFrame()->url();
GURL full_url = current_url.Resolve(args[0].ToString());
string target = "";
@@ -532,8 +532,8 @@
value = args[2].ToString();
// Note: webkit's version does not return the boolean, so neither do we.
- shell_->webView()->GetFocusedFrame()->ExecuteEditCommandByName(command,
- value);
+ shell_->webView()->GetFocusedFrame()->executeCommand(
+ WebString::fromUTF8(command), WebString::fromUTF8(value));
}
result->SetNull();
}
@@ -546,7 +546,8 @@
}
std::string command = args[0].ToString();
- bool rv = shell_->webView()->GetFocusedFrame()->IsEditCommandEnabled(command);
+ bool rv = shell_->webView()->GetFocusedFrame()->isCommandEnabled(
+ WebString::fromUTF8(command));
result->Set(rv);
}
@@ -802,7 +803,7 @@
const CppArgumentList& args, CppVariant* result) {
if (args.size() > 0 && args[0].isString()) {
WebScriptSource source(WebString::fromUTF8(args[0].ToString()));
- shell_->webView()->GetMainFrame()->ExecuteScriptInNewWorld(&source, 1, 1);
+ shell_->webView()->GetMainFrame()->executeScriptInNewWorld(&source, 1, 1);
}
result->SetNull();
}
« no previous file with comments | « webkit/glue/webworker_impl.cc ('k') | webkit/tools/test_shell/plugin_tests.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698