New issue
Advanced search Search tips

Issue 807153 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

LazyBackgroundPageApiTest.UpdateExtensionsPage_MD is Flaky

Project Member Reported by Findit, Jan 30 2018

Issue description

Components: UI>Browser>ExtensionsManagement
Labels: -Pri-1 Pri-2
Owner: dpa...@chromium.org
Flakiness dashboard:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=LazyBackgroundPageApiTest.UpdateExtensionsPage_MD

The failure log is as follows.
--
[ RUN      ] LazyBackgroundPageApiTest.UpdateExtensionsPage_MD
[74067:14855:0129/180906.342888:WARNING:notification_platform_bridge_mac.mm(510)] AlertNotificationService: XPC connection invalidated.
[74067:775:0129/180908.119063:INFO:CONSOLE(0)] "[SUCCESS] (no test)", source: chrome-extension://inkpjbgijlhkidoiccmmkaifooienfoj/_generated_background_page.html (0)
../../chrome/browser/extensions/lazy_background_page_apitest.cc:699: Failure
Value of: is_inactive
  Actual: false
Expected: true

--

The patch added the test, so findit seems to have found the correct CL.
Assigned to the owner of that CL and let me revert the patch since the patch just adds the flaky browser test.
Project Member

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

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

commit 243daae82c765221c4ed7f344bf66d2698136b1d
Author: Makoto Shimazu <shimazu@chromium.org>
Date: Tue Jan 30 05:23:04 2018

Revert "MD Extensions: Port LazyBackgroundPageApiTest.UpdateExtensionsPage to new UI."

This reverts commit 8d170b8707234ed6415a470c39aeb43a396cdd53.

Reason for revert: The test added by this CL is flaky.

Original change's description:
> MD Extensions: Port LazyBackgroundPageApiTest.UpdateExtensionsPage to new UI.
> 
> Bug:  800502 
> Change-Id: I059c10360312df02b040e3e87667b8a39abce09c
> Reviewed-on: https://chromium-review.googlesource.com/891627
> Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#532714}

TBR=rdevlin.cronin@chromium.org,dpapad@chromium.org

Change-Id: I58b2acc0496a3c901e57fc240d96059ff50248ed
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  800502 ,  807153 
Reviewed-on: https://chromium-review.googlesource.com/892658
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532772}
[modify] https://crrev.com/243daae82c765221c4ed7f344bf66d2698136b1d/chrome/browser/extensions/lazy_background_page_apitest.cc

Labels: -Sheriff-Chromium
Removed the sheriff flag since it's reverted.

Comment 4 by dpa...@chromium.org, Jan 30 2018

Owner: shimazu@chromium.org
Status: Fixed (was: Available)
Closing this bug since the CL was reverted. Will try to deflake and re-land as part of  issue 800502 .
 Issue 807822  has been merged into this issue.
Project Member

Comment 6 by Findit, Feb 1 2018

Findit identified the culprit r533394 with confidence 100.0% in the config "tryserver.chromium.mac / mac_chromium_rel_ng"
based on the flakiness trend:

https://findit-for-me.appspot.com/waterfall/flake?key=ag9zfmZpbmRpdC1mb3ItbWVyqQELEhdNYXN0ZXJGbGFrZUFuYWx5c2lzUm9vdCJzY2hyb21pdW0ubWFjL01hYzEwLjkgVGVzdHMvNTM0MjEvYnJvd3Nlcl90ZXN0cy9UR0Y2ZVVKaFkydG5jbTkxYm1SUVlXZGxRWEJwVkdWemRDNVZjR1JoZEdWRmVIUmxibk5wYjI1elVHRm5aVjlOUkE9PQwLEhNNYXN0ZXJGbGFrZUFuYWx5c2lzGAEM

Automatically posted by the findit-for-me app (https://goo.gl/Ot9f7N).
Flake Analyzer is in alpha version.
Feedback is welcome using component Tools>Test>FindIt>Flakiness !

Sign in to add a comment