Skip to content

Commit

Permalink
Disable tests in ContextMenuTest
Browse files Browse the repository at this point in the history
This disables a couple of flaky tests in
org.chromium.chrome.browser.contextmenu.ContextMenuTest:

 - testContextMenuRetrievesVideoOptions
 - testSaveVideo

[email protected]

Bug: 947695
Change-Id: I432f457aeba58b94be4e7a554e63d27e2fd821dc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546806
Reviewed-by: Makoto Shimazu <[email protected]>
Commit-Queue: Makoto Shimazu <[email protected]>
Cr-Commit-Position: refs/heads/master@{#646202}
  • Loading branch information
makotoshimazu authored and Commit Bot committed Apr 1, 2019
1 parent bd5e5a7 commit 93efa83
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@

import org.chromium.base.test.util.CallbackHelper;
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.RetryOnFailure;
Expand Down Expand Up @@ -284,6 +285,7 @@ public void testSaveImage()
@LargeTest
@Feature({"Browser"})
@RetryOnFailure
@DisabledTest(message = "https://crbug.com/947695")
public void testSaveVideo()
throws InterruptedException, TimeoutException, SecurityException, IOException {
// Click the video to enable playback
Expand Down Expand Up @@ -419,6 +421,7 @@ public void testContextMenuRetrievesImageLinkOptions()
@SmallTest
@Feature({"Browser", "ContextMenu"})
@RetryOnFailure
@DisabledTest(message = "https://crbug.com/947695")
public void testContextMenuRetrievesVideoOptions()
throws TimeoutException, InterruptedException {
Tab tab = mDownloadTestRule.getActivity().getActivityTab();
Expand Down

0 comments on commit 93efa83

Please sign in to comment.