MHTMLGenerationTest.ViewedMHTML* tests fail on Android under some circumstances |
|||||||||||||||
Issue description... in particular, on https://build.chromium.org/p/chromium.linux/builders/Android%20Tests (starting with https://build.chromium.org/p/chromium.linux/builders/Android%20Tests/builds/26636, the *second* build after https://codereview.chromium.org/1977303003 landed) and on linux_android_rel_ng. It's not currently being surfaced thanks to https://bugs.chromium.org/p/chromium/issues/detail?id=615086. I've been looking at this for most of the afternoon and am not sure what's going on. I'm unable to reproduce the failure with a local build w/ the same gn args as linux_android_rel_ng but can consistently reproduce it with the build artifact from the trybot. Logcat shows a DCHECK failure (https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Source/core/editing/iterators/SearchBuffer.cpp&l=55), and the symbolized tombstone shows the same thing: https://drive.google.com/open?id=0B-7nYP91qHPhRUFIcU1pdlJ2Ylk I'm going to disable the two tests on Android for now s.t. I can fix issue 615086 and hopefully prevent tests with similar failure modes from landing.
,
May 27 2016
,
May 31 2016
I've been seeing this same problem with other tests too. FindRequestManagerTest.* tests fail on this DCHECK on the linux_android_rel_ng trybot, and ONLY on that trybot, and only sometimes. They were disabled for Android because of it. I haven't been able to reproduce the problem locally no matter what I try (I've tried lots of different configurations and different Android devices). yosin@ has been looking into this as well.
,
May 31 2016
,
Jun 2 2016
,
Jun 3 2016
After talking to yosin@, this is suspected to be a problem with the bot's config for ICU.
,
Jun 3 2016
,
Jun 15 2016
Based on #6, I think someone else is likely to be a better owner of the root cause than me. Paul, is that someone you?
,
Jun 16 2016
No, not me. I looked into this a bit just because I had a test that was failing but I still really don't k ow what's going on here. yosin@ owns the bit of code where the DCHECK is hit, so perhaps he would be a good owner.
,
Jul 6 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cc9a00f96ebe2ec23c1626e5e42de5313e968ef7 commit cc9a00f96ebe2ec23c1626e5e42de5313e968ef7 Author: paulmeyer <paulmeyer@chromium.org> Date: Wed Jul 06 13:44:27 2016 Small optimization in disabling tests on Android Release in find_request_manager_browsertest.cc. This patch removes the need to add MAYBE lines for nearly every test that goes into find_request_manager_browsertest.cc. It does this by defining a MAYBE() function instead of MAYBE/DISABLED definitions for every test name. BUG=615291 Review-Url: https://codereview.chromium.org/2122763002 Cr-Commit-Position: refs/heads/master@{#403898} [modify] https://crrev.com/cc9a00f96ebe2ec23c1626e5e42de5313e968ef7/content/browser/find_request_manager_browsertest.cc
,
Oct 25 2016
Issue 657331 has been merged into this issue.
,
Oct 25 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/65fee8192a9f0c1c1e1c8b8f99b6e046441e71e6 commit 65fee8192a9f0c1c1e1c8b8f99b6e046441e71e6 Author: torne <torne@chromium.org> Date: Tue Oct 25 12:09:03 2016 Disable FindRequestManager tests on android debug also. Tests were already disabled for release; disable for debug as well since the issue doesn't seem to be debug/release specific (possibly test device config is the issue?) BUG=615291 TBR=paulmeyer@chromium.org Review-Url: https://codereview.chromium.org/2449733003 Cr-Commit-Position: refs/heads/master@{#427324} [modify] https://crrev.com/65fee8192a9f0c1c1e1c8b8f99b6e046441e71e6/content/browser/find_request_manager_browsertest.cc
,
Oct 25 2016
,
Nov 28 2016
,
Jan 18 2017
If this is really a Pri-1, find an owner and update the priority. This is the result of a bulk edit that moved high priority available bugs to a lower priority in an attempt to be more honest with bug filers.
,
Jan 24 2018
,
Jan 24 2018
,
Feb 10 2018
yosin@, did you ever determine the cause or confirm the theory in #6?
,
Jun 17 2018
|
|||||||||||||||
►
Sign in to add a comment |
|||||||||||||||
Comment 1 by bugdroid1@chromium.org
, May 27 2016