Commit 6b35c73f authored by petewil@chromium.org's avatar petewil@chromium.org

Disable failing unit test

The PrerenderOmniboxAbandon test fails with a command line that is too long
for the windows RegisterApplicationRestart API.

TBR=mmenke@chromium.org,gavinp@chromium.org

NOTRY=true

BUG=394592

Review URL: https://codereview.chromium.org/400493004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283599 0039d316-1c4b-4281-b951-d872f2087c98
parent 02436fb8
...@@ -4380,8 +4380,18 @@ IN_PROC_BROWSER_TEST_F(PrerenderOmniboxBrowserTest, PrerenderOmniboxCancel) { ...@@ -4380,8 +4380,18 @@ IN_PROC_BROWSER_TEST_F(PrerenderOmniboxBrowserTest, PrerenderOmniboxCancel) {
prerender->WaitForStop(); prerender->WaitForStop();
} }
// Crashes on windows. The failure seems to be that the PrerenderOmniboxAbandon
// test is setting up a call to the windows API RegisterApplicationRestart with
// a command line that is too long (> 1024 characters).
#if defined(OS_WIN) || defined(OS_MACOSX)
#define MAYBE_PrerenderOmniboxAbandon DISABLED_PrerenderOmniboxAbandon
#else
#define MAYBE_PrerenderOmniboxAbandon PrerenderOmniboxAbandon
#endif
// Checks that accepting omnibox input abandons an omnibox prerender. // Checks that accepting omnibox input abandons an omnibox prerender.
IN_PROC_BROWSER_TEST_F(PrerenderOmniboxBrowserTest, PrerenderOmniboxAbandon) { IN_PROC_BROWSER_TEST_F(PrerenderOmniboxBrowserTest,
MAYBE_PrerenderOmniboxAbandon) {
// Set the abandon timeout to something high so it does not introduce // Set the abandon timeout to something high so it does not introduce
// flakiness if the prerender times out before the test completes. // flakiness if the prerender times out before the test completes.
GetPrerenderManager()->mutable_config().abandon_time_to_live = GetPrerenderManager()->mutable_config().abandon_time_to_live =
......
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