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

Unified Diff: chrome/browser/extensions/extension_keybinding_apitest.cc

Issue 11728003: Change ExecuteJavaScript* helper functions in browser_test_utils.{h,cc} (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Adding files for gpu_tests and NaCl browser tests. Created 7 years, 12 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: chrome/browser/extensions/extension_keybinding_apitest.cc
diff --git a/chrome/browser/extensions/extension_keybinding_apitest.cc b/chrome/browser/extensions/extension_keybinding_apitest.cc
index fd04a816cebd49b64feb947c0d13293260779a83..32ed8a00ffd2c00336f7d4552e2743e7ac8a0180 100644
--- a/chrome/browser/extensions/extension_keybinding_apitest.cc
+++ b/chrome/browser/extensions/extension_keybinding_apitest.cc
@@ -90,10 +90,11 @@ IN_PROC_BROWSER_TEST_F(CommandsApiTest, Basic) {
// Verify the command worked.
bool result = false;
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), L"",
- L"setInterval(function(){"
- L" if(document.body.bgColor == 'red'){"
- L" window.domAutomationController.send(true)}}, 100)",
+ tab->GetRenderViewHost(),
+ "",
+ "setInterval(function(){"
+ " if(document.body.bgColor == 'red'){"
+ " window.domAutomationController.send(true)}}, 100)",
&result));
ASSERT_TRUE(result);
@@ -103,10 +104,11 @@ IN_PROC_BROWSER_TEST_F(CommandsApiTest, Basic) {
result = false;
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), L"",
- L"setInterval(function(){"
- L" if(document.body.bgColor == 'blue'){"
- L" window.domAutomationController.send(true)}}, 100)",
+ tab->GetRenderViewHost(),
+ "",
+ "setInterval(function(){"
+ " if(document.body.bgColor == 'blue'){"
+ " window.domAutomationController.send(true)}}, 100)",
&result));
ASSERT_TRUE(result);
}
@@ -150,10 +152,11 @@ IN_PROC_BROWSER_TEST_F(CommandsApiTest, MAYBE_PageAction) {
WebContents* tab = chrome::GetActiveWebContents(browser());
bool result = false;
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), L"",
- L"setInterval(function(){"
- L" if(document.body.bgColor == 'red'){"
- L" window.domAutomationController.send(true)}}, 100)",
+ tab->GetRenderViewHost(),
+ "",
+ "setInterval(function(){"
+ " if(document.body.bgColor == 'red'){"
+ " window.domAutomationController.send(true)}}, 100)",
&result));
ASSERT_TRUE(result);
}
« no previous file with comments | « chrome/browser/extensions/extension_incognito_apitest.cc ('k') | chrome/browser/extensions/extension_nacl_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698