webkit_tests failing on chromium.webkit/WebKit Android (Nexus4) |
|||||
Issue descriptionwebkit_tests failing on chromium.webkit/WebKit Android (Nexus4) Type: build-failure Builders failed on: - WebKit Android (Nexus4): https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29 this has been failing consistently for a couple days with no obvious culprit (even from the initial blamelist)
,
Nov 17 2016
Hm, it looks like the only CLs in third_party/WebKit were mine, although I believe that nothing committed in that range should have affected layout test results; two CLs in that range that mention android are: r432309 Replace the use of WindowAndroid with Tab in permissions code. r432310 Android: support multiple displays on C++ side
,
Nov 17 2016
My r432314 only changed browser tests - it shouldn't have affected content_shell and/or layout tests.
,
Nov 17 2016
I'm pretty sure that my OWNERS file change was unrelated.
,
Nov 17 2016
The failing tests don't look like they invoke a permission prompt, which was my Android change.
,
Nov 17 2016
r432310 Android: support multiple displays on C++ side could lead to failure if this is a crash.
,
Nov 18 2016
timav, do you mind reverting to see if it resolves the issue?
,
Nov 18 2016
Created revert in https://codereview.chromium.org/2514633002/
,
Nov 18 2016
The revert landed: commit ece04082b79fd30261b48ad895e7f05beeda7823 Author: timav <timav@chromium.org> Date: Fri Nov 18 04:38:33 2016 Cr-Commit-Position: refs/heads/master@{#433108} Will keep an eye on this.
,
Nov 18 2016
Remove from sheriff queue (if it continues to be a problem it will come up again in SOM).
,
Nov 18 2016
,
Nov 18 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by est...@chromium.org
, Nov 17 2016Labels: OS-Android