Index: chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java |
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java |
index 914deec1c5e3ac9884f0ad333a878c86477db3cc..929aa28032a3a852062f8ee1ad1412d4fb13f9ad 100644 |
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java |
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java |
@@ -11,7 +11,6 @@ import android.content.pm.ActivityInfo; |
import android.os.Debug; |
import android.os.Environment; |
import android.os.SystemClock; |
-import android.test.FlakyTest; |
import android.test.suitebuilder.annotation.LargeTest; |
import android.test.suitebuilder.annotation.MediumTest; |
import android.test.suitebuilder.annotation.SmallTest; |
@@ -27,6 +26,7 @@ import org.chromium.base.annotations.SuppressFBWarnings; |
import org.chromium.base.test.util.CommandLineFlags; |
import org.chromium.base.test.util.DisabledTest; |
import org.chromium.base.test.util.Feature; |
+import org.chromium.base.test.util.FlakyTest; |
import org.chromium.base.test.util.Restriction; |
import org.chromium.base.test.util.UrlUtils; |
import org.chromium.chrome.R; |
@@ -409,12 +409,11 @@ public class TabsTest extends ChromeTabbedActivityTestBase { |
* Verify close button works in the TabSwitcher in landscape mode. |
* This code does not handle properly different screen densities. |
* @throws InterruptedException |
- * Bug: crbug.com/170179 |
* @Restriction({RESTRICTION_TYPE_PHONE, RESTRICTION_TYPE_NON_LOW_END_DEVICE}) |
* @LargeTest |
* @Feature({"Android-TabSwitcher"}) |
*/ |
- @FlakyTest |
+ @FlakyTest(message = "crbug.com/170179") |
public void testTabSwitcherLandscapeCloseButton() throws InterruptedException { |
getActivity().setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_LANDSCAPE); |
// Hard-coded coordinates of the close button on the bottom left of the screen. |
@@ -491,9 +490,8 @@ public class TabsTest extends ChromeTabbedActivityTestBase { |
* @EnormousTest |
* @TimeoutScale(30) |
* @Feature({"Navigation"}) |
- * crbug/223110 |
*/ |
- @FlakyTest |
+ @FlakyTest(message = "crbug.com/223110") |
public void testOpenManyTabsAtOnce10() throws InterruptedException { |
openAndVerifyManyTestTabs(10); |
} |
@@ -620,11 +618,10 @@ public class TabsTest extends ChromeTabbedActivityTestBase { |
* Verifies that the tab are actually stacking at the bottom and top of the screen. |
*/ |
/** |
- * Bug: crbug.com/170179 |
* @LargeTest |
* @Feature({"Android-TabSwitcher"}) |
*/ |
- @FlakyTest |
+ @FlakyTest(message = "crbug.com/170179") |
@Restriction({ChromeRestriction.RESTRICTION_TYPE_PHONE, RESTRICTION_TYPE_NON_LOW_END_DEVICE}) |
public void testTabsStacking() throws InterruptedException { |
final int count = openTabs(12, false); |
@@ -686,11 +683,10 @@ public class TabsTest extends ChromeTabbedActivityTestBase { |
* Verify that switching back and forth to the tabswitcher does not leak memory. |
*/ |
/** |
- * Bug: crbug.com/303319 |
* @LargeTest |
* @Feature({"Android-TabSwitcher"}) |
*/ |
- @FlakyTest |
+ @FlakyTest(message = "crbug.com/303319") |
@Restriction(ChromeRestriction.RESTRICTION_TYPE_PHONE) |
public void testTabSwitcherMemoryLeak() throws InterruptedException { |
openTabs(4, true); |
@@ -759,11 +755,10 @@ public class TabsTest extends ChromeTabbedActivityTestBase { |
} |
/** |
- * Bug: crbug.com/170179 |
* @LargeTest |
* @Feature({"Android-TabSwitcher"}) |
*/ |
- @FlakyTest |
+ @FlakyTest(message = "crbug.com/170179") |
@Restriction({ChromeRestriction.RESTRICTION_TYPE_PHONE, RESTRICTION_TYPE_NON_LOW_END_DEVICE}) |
public void testTabSelectionLandscape() throws InterruptedException { |
getActivity().setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_LANDSCAPE); |