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

Side by Side Diff: chrome/browser/prerender/prerender_browsertest.cc

Issue 10778031: Disable PrerenderBrowserTest.PrerenderHTML5Audio under ASan (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 5 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <deque> 5 #include <deque>
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/path_service.h" 8 #include "base/path_service.h"
9 #include "base/string_util.h" 9 #include "base/string_util.h"
10 #include "base/stringprintf.h" 10 #include "base/stringprintf.h"
(...skipping 1597 matching lines...) Expand 10 before | Expand all | Expand 10 after
1608 } 1608 }
1609 EXPECT_EQ(0, num_prerender_tabs); 1609 EXPECT_EQ(0, num_prerender_tabs);
1610 1610
1611 // We may have deleted the prerender tab, but the swapped in tab should be 1611 // We may have deleted the prerender tab, but the swapped in tab should be
1612 // active. 1612 // active.
1613 EXPECT_GE(num_tabs_with_prerender_page_title, 1); 1613 EXPECT_GE(num_tabs_with_prerender_page_title, 1);
1614 EXPECT_LE(num_tabs_with_prerender_page_title, 2); 1614 EXPECT_LE(num_tabs_with_prerender_page_title, 2);
1615 } 1615 }
1616 1616
1617 // Checks that audio loads are deferred on prerendering. 1617 // Checks that audio loads are deferred on prerendering.
1618 IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, PrerenderHTML5Audio) { 1618 // Times out under AddressSanitizer, see http://crbug.com/108402
1619 #if defined(ADDRESS_SANITIZER)
1620 #define MAYBE_PrerenderHTML5Audio DISABLED_PrerenderHTML5Audio
1621 #else
1622 #define MAYBE_PrerenderHTML5Audio PrerenderHTML5Audio
1623 #endif
1624 IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, MAYBE_PrerenderHTML5Audio) {
1619 PrerenderTestURL("files/prerender/prerender_html5_audio.html", 1625 PrerenderTestURL("files/prerender/prerender_html5_audio.html",
1620 FINAL_STATUS_USED, 1626 FINAL_STATUS_USED,
1621 1); 1627 1);
1622 NavigateToDestUrlAndWaitForPassTitle(); 1628 NavigateToDestUrlAndWaitForPassTitle();
1623 } 1629 }
1624 1630
1625 // Checks that audio loads are deferred on prerendering and played back when 1631 // Checks that audio loads are deferred on prerendering and played back when
1626 // the prerender is swapped in if autoplay is set. 1632 // the prerender is swapped in if autoplay is set.
1627 IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, PrerenderHTML5AudioAutoplay) { 1633 IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, PrerenderHTML5AudioAutoplay) {
1628 PrerenderTestURL("files/prerender/prerender_html5_audio_autoplay.html", 1634 PrerenderTestURL("files/prerender/prerender_html5_audio_autoplay.html",
(...skipping 739 matching lines...) Expand 10 before | Expand all | Expand 10 after
2368 ResultCatcher catcher; 2374 ResultCatcher catcher;
2369 2375
2370 PrerenderTestURL("files/prerender/prerender_page.html", FINAL_STATUS_USED, 1); 2376 PrerenderTestURL("files/prerender/prerender_page.html", FINAL_STATUS_USED, 1);
2371 NavigateToDestURL(); 2377 NavigateToDestURL();
2372 ASSERT_TRUE(IsEmptyPrerenderLinkManager()); 2378 ASSERT_TRUE(IsEmptyPrerenderLinkManager());
2373 2379
2374 ASSERT_TRUE(catcher.GetNextResult()) << catcher.message(); 2380 ASSERT_TRUE(catcher.GetNextResult()) << catcher.message();
2375 } 2381 }
2376 2382
2377 } // namespace prerender 2383 } // namespace prerender
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698