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

Issue 832686 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

blink_perf.layout/subtree-detaching.html in blink_perf.layout failing on multiple builders

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Apr 13 2018

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of charliea@chromium.org

blink_perf.layout/subtree-detaching.html in blink_perf.layout failing on multiple builders

Builders failed on: 
- Android Nexus5 Perf: 
  https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus5%20Perf
- Android Nexus7v2 Perf: 
  https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus7v2%20Perf

CCing eae@, the benchmark owner.
 
Based on the failure log (https://chromium-swarm.appspot.com/task?id=3cd6e921afe0e910&refresh=10&show_raw=1) which contains a DevtoolsTargetCrashException and a breakpad dump (https://paste.googleplex.com/5863660404604928), it looks like Chrome is crashing.
Based on the flakiness dashboard, it looks like the failure was introduced between r550299 and r550442.
Cc: nednguyen@chromium.org
Unfortunately, I'm unable to launch a bisect here because it looks like the test was never passing: it started failing as soon as it was added. Looking through the revision range, though, (https://chromium.googlesource.com/chromium/src/+log/d780e1bf31f184f6ab0071b57aba8b0595487023%5E..12b708c733623cf4ce0ef775a77f50288fe6f39d?pretty=fuller&n=), I can't find any evidence of this story being added.

For now, I'm going to disable. nednguyen@, could you look at the revision range to try and see what CL added this test? I don't see any changes to the LayoutTests directory that would have added this test (searching for "subtree" yields nothing).
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 13 2018

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

commit 2ef7b3744f7af7af83b055f2e3d9b37194fcec30
Author: Charlie Andrews <charliea@chromium.org>
Date: Fri Apr 13 17:57:33 2018

Disable failing blink_perf.layout test on Android

TBR=nednguyen@chromium.org
NOTRY=true

Bug: 832686
Change-Id: Ia03388a34d307ddea50cd8158a4e09a2243dfe6e
Reviewed-on: https://chromium-review.googlesource.com/1012483
Reviewed-by: Charlie Andrews <charliea@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550678}
[modify] https://crrev.com/2ef7b3744f7af7af83b055f2e3d9b37194fcec30/tools/perf/expectations.config

Components: Blink>Layout
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 17 2018

Labels: merge-merged-testbranch
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2ef7b3744f7af7af83b055f2e3d9b37194fcec30

commit 2ef7b3744f7af7af83b055f2e3d9b37194fcec30
Author: Charlie Andrews <charliea@chromium.org>
Date: Fri Apr 13 17:57:33 2018

Disable failing blink_perf.layout test on Android

TBR=nednguyen@chromium.org
NOTRY=true

Bug: 832686
Change-Id: Ia03388a34d307ddea50cd8158a4e09a2243dfe6e
Reviewed-on: https://chromium-review.googlesource.com/1012483
Reviewed-by: Charlie Andrews <charliea@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550678}
[modify] https://crrev.com/2ef7b3744f7af7af83b055f2e3d9b37194fcec30/tools/perf/expectations.config

Sign in to add a comment