Commit cd431217 authored by sky@chromium.org's avatar sky@chromium.org

Removes FLAKY from a couple of tests as they have not failed in the

last 300 or so times. They are crashing on chromeos though, but that
is logged as 48911.

BUG=39925 54894
TEST=none

Review URL: http://codereview.chromium.org/6372015

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72727 0039d316-1c4b-4281-b951-d872f2087c98
parent 70b1f803
...@@ -24,13 +24,17 @@ ...@@ -24,13 +24,17 @@
#define MAYBE_BasicRestoreFromClosedWindow BasicRestoreFromClosedWindow #define MAYBE_BasicRestoreFromClosedWindow BasicRestoreFromClosedWindow
#endif #endif
// http://crbug.com/ // http://crbug.com/48911
#if defined(OS_CHROMEOS) && !defined(NDEBUG) #if defined(OS_CHROMEOS) && !defined(NDEBUG)
#define MAYBE_RestoreWindowAndTab DISABLED_RestoreWindowAndTab #define MAYBE_RestoreWindowAndTab DISABLED_RestoreWindowAndTab
#define MAYBE_RestoreWindow DISABLED_RestoreWindow #define MAYBE_RestoreWindow DISABLED_RestoreWindow
#define MAYBE_RestoreToDifferentWindow DISABLED_RestoreToDifferentWindow
#define MAYBE_RestoreIntoSameWindow DISABLED_RestoreIntoSameWindow
#else #else
#define MAYBE_RestoreWindowAndTab RestoreWindowAndTab #define MAYBE_RestoreWindowAndTab RestoreWindowAndTab
#define MAYBE_RestoreWindow RestoreWindow #define MAYBE_RestoreWindow RestoreWindow
#define MAYBE_RestoreToDifferentWindow RestoreToDifferentWindow
#define MAYBE_RestoreIntoSameWindow RestoreIntoSameWindow
#endif #endif
class TabRestoreUITest : public UITest { class TabRestoreUITest : public UITest {
...@@ -207,8 +211,7 @@ TEST_F(TabRestoreUITest, MiddleTab) { ...@@ -207,8 +211,7 @@ TEST_F(TabRestoreUITest, MiddleTab) {
// Close a tab, switch windows, then restore the tab. The tab should be in its // Close a tab, switch windows, then restore the tab. The tab should be in its
// original window and position, and active. // original window and position, and active.
// This test is flaky. See http://crbug.com/54894 TEST_F(TabRestoreUITest, MAYBE_RestoreToDifferentWindow) {
TEST_F(TabRestoreUITest, FLAKY_RestoreToDifferentWindow) {
scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0)); scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0));
ASSERT_TRUE(browser_proxy.get()); ASSERT_TRUE(browser_proxy.get());
...@@ -396,8 +399,7 @@ TEST_F(TabRestoreUITest, MAYBE_RestoreWindowAndTab) { ...@@ -396,8 +399,7 @@ TEST_F(TabRestoreUITest, MAYBE_RestoreWindowAndTab) {
// Open a window with two tabs, close both (closing the window), then restore // Open a window with two tabs, close both (closing the window), then restore
// both. Make sure both restored tabs are in the same window. // both. Make sure both restored tabs are in the same window.
// http://crbug.com/39925 TEST_F(TabRestoreUITest, MAYBE_RestoreIntoSameWindow) {
TEST_F(TabRestoreUITest, FLAKY_RestoreIntoSameWindow) {
scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0)); scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0));
ASSERT_TRUE(browser_proxy.get()); ASSERT_TRUE(browser_proxy.get());
CheckActiveWindow(browser_proxy.get()); CheckActiveWindow(browser_proxy.get());
......
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