New issue
Advanced search Search tips

Issue 883680 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 3
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug
Flaky-Test: AndroidUsbCountTest.TestNoMultipleCallsRemoveInCallback



Sign in to add a comment

AndroidUsbCountTest.TestNoMultipleCallsRemoveInCallback is flaky

Project Member Reported by Findit, Sep 13

Issue description

Owner: vkuzkokov@chromium.org
Status: Assigned (was: Untriaged)
Labels: -Sheriff-Chromium
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 13

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

commit e3170c4fc248af87dee30af2acc06767d1e3a936
Author: Oskar Sundbom <ossu@chromium.org>
Date: Thu Sep 13 13:40:11 2018

Disable AndroidUsbCountTest.TestNoMultipleCallsRemoveAddInCallback

It's been flaky for a long time.

Bug:  883680 
Tbr: pfeldman@chromium.org
Change-Id: Id9d37b76480dab89325d2d454f921bb76288fb4d
Reviewed-on: https://chromium-review.googlesource.com/1224551
Reviewed-by: Oskar Sundbom <ossu@chromium.org>
Commit-Queue: Oskar Sundbom <ossu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590991}
[modify] https://crrev.com/e3170c4fc248af87dee30af2acc06767d1e3a936/chrome/browser/devtools/device/usb/android_usb_browsertest.cc

Components: Platform>DevTools
Owner: dgozman@chromium.org
Not working on DevTools anymore. FWIW, any test that uses IsIdleForTesting is bound to be flaky to some extent.
Status: Archived (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 12

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

commit 2b4cda586f3e85c9c86b36aeebf07f8a984b3953
Author: Xiyuan Xia <xiyuan@chromium.org>
Date: Wed Dec 12 17:24:05 2018

Remove MessageLoopCurrent::IsIdleForTesting in AndroidUsbCountTest

Testing of message loop idle seems not relevant to the tests
and is bound to be flaky in browser_tests.

Bug:  914380 ,  883680 
Change-Id: Iac6862c995ae105b8451f0cbcb544f64473c0d85
Reviewed-on: https://chromium-review.googlesource.com/c/1374199
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#615938}
[modify] https://crrev.com/2b4cda586f3e85c9c86b36aeebf07f8a984b3953/chrome/browser/devtools/device/usb/android_usb_browsertest.cc

Status: Fixed (was: Archived)

Sign in to add a comment