Commit a9f854df authored by David Bertoni's avatar David Bertoni Committed by Commit Bot

[Extensions] Remove obsolete test.

The ExtensionApiTest.ContentScriptStylesInjectedIntoExistingRenderers
was disabled on 2012-06-22 for being flaky. It is no longer a valid
test.

Bug: 120762
Change-Id: Id3b89ec7e04a83c1b38cea3d671672b7ca3277c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2132670Reviewed-by: default avatarKelvin Jiang <kelvinjiang@chromium.org>
Commit-Queue: David Bertoni <dbertoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#756807}
parent ecf1df6d
...@@ -326,36 +326,6 @@ IN_PROC_BROWSER_TEST_F(ContentScriptCssInjectionTest, ...@@ -326,36 +326,6 @@ IN_PROC_BROWSER_TEST_F(ContentScriptCssInjectionTest,
EXPECT_TRUE(CheckStyleInjection(browser(), url, false)); EXPECT_TRUE(CheckStyleInjection(browser(), url, false));
} }
// crbug.com/120762
IN_PROC_BROWSER_TEST_F(
ExtensionApiTest,
DISABLED_ContentScriptStylesInjectedIntoExistingRenderers) {
ASSERT_TRUE(StartEmbeddedTestServer());
content::WindowedNotificationObserver signal(
extensions::NOTIFICATION_USER_SCRIPTS_UPDATED,
content::Source<Profile>(browser()->profile()));
// Start with a renderer already open at a URL.
GURL url(embedded_test_server()->GetURL("/extensions/test_file.html"));
ui_test_utils::NavigateToURL(browser(), url);
LoadExtension(
test_data_dir_.AppendASCII("content_scripts/existing_renderers"));
signal.Wait();
// And check that its styles were affected by the styles that just got loaded.
bool styles_injected;
ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
browser()->tab_strip_model()->GetActiveWebContents(),
"window.domAutomationController.send("
" document.defaultView.getComputedStyle(document.body, null)."
" getPropertyValue('background-color') == 'rgb(255, 0, 0)')",
&styles_injected));
ASSERT_TRUE(styles_injected);
}
IN_PROC_BROWSER_TEST_F(ContentScriptApiTest, ContentScriptCSSLocalization) { IN_PROC_BROWSER_TEST_F(ContentScriptApiTest, ContentScriptCSSLocalization) {
ASSERT_TRUE(StartEmbeddedTestServer()); ASSERT_TRUE(StartEmbeddedTestServer());
ASSERT_TRUE(RunExtensionTest("content_scripts/css_l10n")) << message_; ASSERT_TRUE(RunExtensionTest("content_scripts/css_l10n")) << message_;
......
{
"name": "Existing Renderers",
"version": "1.0",
"manifest_version": 2,
"description": "Tests that styles specified by content scripts are injected and applied into existing renderers.",
"content_scripts": [ {
"css": [ "content_styles.css" ],
"matches": [ "<all_urls>" ]
}]
}
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment