Commit 0e15a031 authored by Hayato Ito's avatar Hayato Ito Committed by Commit Bot

Disable DeclarativeContentApiTest.RemoveAllRulesAfterExtensionUninstall

Disable DeclarativeContentApiTest.RemoveAllRulesAfterExtensionUninstall
on all platforms

TBR=tbarzic@chromium.org

BUG: 1013457
Change-Id: Ib0a3f02a340b35072d4572d22e2a7c5f27088348
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855239Reviewed-by: default avatarHayato Ito <hayato@chromium.org>
Commit-Queue: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704941}
parent d0191ba1
...@@ -967,16 +967,10 @@ IN_PROC_BROWSER_TEST_F(DeclarativeContentApiTest, ...@@ -967,16 +967,10 @@ IN_PROC_BROWSER_TEST_F(DeclarativeContentApiTest,
} }
// https://crbug.com/517492 // https://crbug.com/517492
#if defined(OS_WIN)
// Fails on XP: http://crbug.com/515717 // Fails on XP: http://crbug.com/515717
#define MAYBE_RemoveAllRulesAfterExtensionUninstall \ // Fails on other platfomrs: http://crbug.com/1013457
DISABLED_RemoveAllRulesAfterExtensionUninstall
#else
#define MAYBE_RemoveAllRulesAfterExtensionUninstall \
RemoveAllRulesAfterExtensionUninstall
#endif
IN_PROC_BROWSER_TEST_F(DeclarativeContentApiTest, IN_PROC_BROWSER_TEST_F(DeclarativeContentApiTest,
MAYBE_RemoveAllRulesAfterExtensionUninstall) { DISABLED_RemoveAllRulesAfterExtensionUninstall) {
ext_dir_.WriteManifest(kDeclarativeContentManifest); ext_dir_.WriteManifest(kDeclarativeContentManifest);
ext_dir_.WriteFile(FILE_PATH_LITERAL("background.js"), kBackgroundHelpers); ext_dir_.WriteFile(FILE_PATH_LITERAL("background.js"), kBackgroundHelpers);
...@@ -1015,7 +1009,6 @@ IN_PROC_BROWSER_TEST_F(DeclarativeContentApiTest, ...@@ -1015,7 +1009,6 @@ IN_PROC_BROWSER_TEST_F(DeclarativeContentApiTest,
ExecuteScriptInBackgroundPage(extension->id(), kRemoveTestRule1)); ExecuteScriptInBackgroundPage(extension->id(), kRemoveTestRule1));
} }
// TODO(wittman): Once ChromeContentRulesRegistry operates on condition and // TODO(wittman): Once ChromeContentRulesRegistry operates on condition and
// action interfaces, add a test that checks that a navigation always evaluates // action interfaces, add a test that checks that a navigation always evaluates
// consistent URL state for all conditions. i.e.: if condition1 evaluates to // consistent URL state for all conditions. i.e.: if condition1 evaluates to
......
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