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

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

Issue 11734018: Revert 174880 due to compilation error in pdf_browsertest.cc (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1374/src/
Patch Set: 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/plugin_apitest.cc
===================================================================
--- chrome/browser/extensions/plugin_apitest.cc (revision 174946)
+++ chrome/browser/extensions/plugin_apitest.cc (working copy)
@@ -48,7 +48,7 @@
// With no extensions, the plugin should not be loaded.
bool result = false;
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), "", "testPluginWorks()", &result));
+ tab->GetRenderViewHost(), L"", L"testPluginWorks()", &result));
EXPECT_FALSE(result);
ExtensionService* service = extensions::ExtensionSystem::Get(
@@ -60,7 +60,7 @@
EXPECT_EQ(size_before + 1, service->extensions()->size());
// Now the plugin should be in the cache.
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), "", "testPluginWorks()", &result));
+ tab->GetRenderViewHost(), L"", L"testPluginWorks()", &result));
// We don't allow extension plugins to run on ChromeOS.
#if defined(OS_CHROMEOS)
EXPECT_FALSE(result);
@@ -75,7 +75,7 @@
// Now the plugin should be unloaded, and the page should be broken.
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), "", "testPluginWorks()", &result));
+ tab->GetRenderViewHost(), L"", L"testPluginWorks()", &result));
EXPECT_FALSE(result);
// If we reload the extension and page, it should work again.
@@ -91,7 +91,7 @@
observer.Wait();
}
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), "", "testPluginWorks()", &result));
+ tab->GetRenderViewHost(), L"", L"testPluginWorks()", &result));
// We don't allow extension plugins to run on ChromeOS.
#if defined(OS_CHROMEOS)
EXPECT_FALSE(result);
@@ -125,7 +125,7 @@
WebContents* tab = chrome::GetActiveWebContents(browser());
bool result = false;
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), "", "testPluginWorks()", &result));
+ tab->GetRenderViewHost(), L"", L"testPluginWorks()", &result));
// We don't allow extension plugins to run on ChromeOS.
#if defined(OS_CHROMEOS)
EXPECT_FALSE(result);
@@ -142,7 +142,7 @@
browser()->profile()->GetHostContentSettingsMap()->SetDefaultContentSetting(
CONTENT_SETTINGS_TYPE_PLUGINS, CONTENT_SETTING_BLOCK);
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), "", "testPluginWorks()", &result));
+ tab->GetRenderViewHost(), L"", L"testPluginWorks()", &result));
// We don't allow extension plugins to run on ChromeOS.
#if defined(OS_CHROMEOS)
EXPECT_FALSE(result);
@@ -154,6 +154,6 @@
ui_test_utils::NavigateToURL(browser(),
net::FilePathToFileURL(extension_dir.AppendASCII("test.html")));
ASSERT_TRUE(content::ExecuteJavaScriptAndExtractBool(
- tab->GetRenderViewHost(), "", "testPluginWorks()", &result));
+ tab->GetRenderViewHost(), L"", L"testPluginWorks()", &result));
EXPECT_FALSE(result);
}
« no previous file with comments | « chrome/browser/extensions/options_page_apitest.cc ('k') | chrome/browser/extensions/subscribe_page_action_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698