New issue
Advanced search Search tips

Issue 764691 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

ContextMenuTestCase.testContextMenuOpenInNewTab is flaky

Project Member Reported by rohitrao@chromium.org, Sep 13 2017

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Sep 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4133e72f27220bb2d6dda56edd4a541c9212d527

commit 4133e72f27220bb2d6dda56edd4a541c9212d527
Author: Rohit Rao <rohitrao@google.com>
Date: Wed Sep 13 13:22:10 2017

[ios] Disables testContextMenuOpenInNewTab on iOS 11.

This test is failing flakily on the bots.

BUG= 764691 

Change-Id: Ief33f7d5613a12610d6734db0ce798ac7564644c
Reviewed-on: https://chromium-review.googlesource.com/664957
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Commit-Queue: Rohit Rao (ping after 24h) <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501609}
[modify] https://crrev.com/4133e72f27220bb2d6dda56edd4a541c9212d527/ios/chrome/browser/context_menu/context_menu_egtest.mm

Comment 2 by sczs@chromium.org, Sep 13 2017

Status: Assigned (was: Untriaged)
eugenebut@ could you PTAL. 
Components: Tests>Disabled
Labels: -Pri-3 Pri-2
Components: Tests>Disabled
Labels: Test-Disabled
Components: -Mobile>WebView>Glue -Tests>Disabled UI>Browser>Core
Project Member

Comment 7 by bugdroid1@chromium.org, Jan 30 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/850b6d3fd8dd507d2b0d51d5435f5bfea2c1615c

commit 850b6d3fd8dd507d2b0d51d5435f5bfea2c1615c
Author: Eugene But <eugenebut@google.com>
Date: Tue Jan 30 00:33:58 2018

Enable testContextMenuOpenInNewTab.

The original failure reason is not included in the bug.
If the test was only  failing on device then it was not
disabled correctly.

The test has passed twice on the bots, so enabling it
to diagnose the problem.

Bug:  764691 
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ie6787ca7124171c75d885322f77b124276723886
Reviewed-on: https://chromium-review.googlesource.com/892083
Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532697}
[modify] https://crrev.com/850b6d3fd8dd507d2b0d51d5435f5bfea2c1615c/ios/chrome/browser/context_menu/context_menu_egtest.mm

Status: Fixed (was: Assigned)
Does not fail anymore according to Flakiness Dashboard

Sign in to add a comment