New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 710662 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Disable flaky showcase tab tests

Project Member Reported by liaoyuke@chromium.org, Apr 11 2017

Issue description

Disabling following tests because they are flaky on devices (failed 5 times during the past 20 runs):

SCTabTestCase/testLaunchWithTopToolbar
SCTabTestCase/testLaunchWithBottomToolbar
SCTabTestCase/testLaunchAndTappingCell
 
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 11 2017

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

commit 0a3ff882814b1367aa8397337e2696686816efd4
Author: liaoyuke <liaoyuke@chromium.org>
Date: Tue Apr 11 23:22:54 2017

disable flaky showcase tests

Disabling following tests because they are flaky on devices (failed 5
times during the past 20 runs):

SCTabTestCase/testLaunchWithTopToolbar
SCTabTestCase/testLaunchWithBottomToolbar
SCTabTestCase/testLaunchAndTappingCell

TBR=lpromero@chromium.org
BUG= 710662 

Review-Url: https://codereview.chromium.org/2815773002
Cr-Commit-Position: refs/heads/master@{#463831}

[modify] https://crrev.com/0a3ff882814b1367aa8397337e2696686816efd4/ios/showcase/tab/sc_tab_egtest.mm
[modify] https://crrev.com/0a3ff882814b1367aa8397337e2696686816efd4/ios/showcase/tab_grid/sc_tab_grid_egtest.mm

Blockedon: -710385

Comment 4 by cma...@chromium.org, Apr 13 2017

Hello Ed, 710385 is now fixed. Can you retake a look at these showcase tab tests?

Comment 5 by edchin@chromium.org, Apr 13 2017

Yuke gave me some very good clues to this issue. Apparently 710385 was not
fixed. I am now actively working on solving this.

Comment 6 by cma...@chromium.org, Apr 17 2017

Great!

Comment 7 by cma...@chromium.org, Apr 17 2017

Status: Started (was: Assigned)

Comment 8 by cma...@chromium.org, Apr 20 2017

Labels: -ReleaseBlock-Beta
This is a showcase test and therefore it is not an RBB
Components: -Tests>Disabled -Test>iOS Tests>Flaky
Project Member

Comment 10 by bugdroid1@chromium.org, Nov 16 2017

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

commit 71fadc89028fa77f71391e64361ea6c37372f53c
Author: Mark Cogan <marq@google.com>
Date: Thu Nov 16 18:35:41 2017

[iOS showcase] Remove dependencies on clean/ from showcase.

This CL removes clean/ view controllers from Showcase.

Bug:  785925 ,  710662 ,  772186 
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Iebce9f69b7776aa2755e46ec379d7245075a55a7
Reviewed-on: https://chromium-review.googlesource.com/774465
Commit-Queue: Mark Cogan <marq@chromium.org>
Reviewed-by: Louis Romero <lpromero@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517135}
[modify] https://crrev.com/71fadc89028fa77f71391e64361ea6c37372f53c/ios/showcase/BUILD.gn
[modify] https://crrev.com/71fadc89028fa77f71391e64361ea6c37372f53c/ios/showcase/DEPS
[modify] https://crrev.com/71fadc89028fa77f71391e64361ea6c37372f53c/ios/showcase/core/showcase_model.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/ntp/BUILD.gn
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/ntp/sc_ntp_coordinator.h
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/ntp/sc_ntp_coordinator.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/root/BUILD.gn
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/root/sc_root_coordinator.h
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/root/sc_root_coordinator.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/root/sc_root_egtest.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab/BUILD.gn
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab/sc_bottom_toolbar_tab_coordinator.h
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab/sc_bottom_toolbar_tab_coordinator.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab/sc_tab_egtest.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab/sc_top_toolbar_tab_coordinator.h
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab/sc_top_toolbar_tab_coordinator.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab_grid/BUILD.gn
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab_grid/sc_tab_grid_coordinator.h
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab_grid/sc_tab_grid_coordinator.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tab_grid/sc_tab_grid_egtest.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/toolbar/BUILD.gn
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/toolbar/sc_toolbar_coordinator.h
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/toolbar/sc_toolbar_coordinator.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/toolbar/sc_toolbar_egtest.mm
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tools_menu/BUILD.gn
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tools_menu/sc_tools_coordinator.h
[delete] https://crrev.com/4935983c64668fb4a3d0e1eb8a4fcc4d3933113f/ios/showcase/tools_menu/sc_tools_coordinator.mm

Status: Fixed (was: Started)
This is no longer relevant as clean was deleted. I'm marking this fixed by virtue of being deleted.

Sign in to add a comment