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

Unified Diff: chrome/browser/ui/webui/webui_webview_browsertest.cc

Issue 1058113002: Implement <webview>.addContentScript/removeContentScript API [3] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@webui_api_1
Patch Set: nits. Created 5 years, 8 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/ui/webui/webui_webview_browsertest.cc
diff --git a/chrome/browser/ui/webui/webui_webview_browsertest.cc b/chrome/browser/ui/webui/webui_webview_browsertest.cc
index 255d47f3f1e7479622cd19fcf312031e2c239156..6c3ed9c8cfcaba3787a706d88cad7b4f43af6b81 100644
--- a/chrome/browser/ui/webui/webui_webview_browsertest.cc
+++ b/chrome/browser/ui/webui/webui_webview_browsertest.cc
@@ -17,7 +17,7 @@ class WebUIWebViewBrowserTest : public WebUIBrowserTest {
void SetUpOnMainThread() override {
WebUIBrowserTest::SetUpOnMainThread();
AddLibrary(
- base::FilePath(FILE_PATH_LITERAL("webview_execute_script_test.js")));
+ base::FilePath(FILE_PATH_LITERAL("webview_content_script_test.js")));
base::FilePath test_data_dir;
PathService::Get(chrome::DIR_TEST_DATA, &test_data_dir);
@@ -123,3 +123,11 @@ IN_PROC_BROWSER_TEST_F(WebUIWebViewBrowserTest,
"testContentScriptExistsAsLongAsWebViewTagExists",
new base::StringValue(GetTestUrl("empty.html").spec())));
}
+
+IN_PROC_BROWSER_TEST_F(WebUIWebViewBrowserTest, AddContentScriptWithCode) {
+ ui_test_utils::NavigateToURL(browser(), GetWebViewEnabledWebUIURL());
+
+ ASSERT_TRUE(WebUIBrowserTest::RunJavascriptAsyncTest(
+ "testAddContentScriptWithCode",
+ new base::StringValue(GetTestUrl("empty.html").spec())));
+}

Powered by Google App Engine
This is Rietveld 408576698