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

Issue 615291 link

Starred by 4 users

Issue metadata

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

Blocking:
issue 457440



Sign in to add a comment

MHTMLGenerationTest.ViewedMHTML* tests fail on Android under some circumstances

Project Member Reported by jbudorick@chromium.org, May 27 2016

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.
 
Cc: -dewittj@chromium.org jbudorick@chromium.org
Owner: dewittj@chromium.org
Cc: yosin@chromium.org
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.
Cc: paulmeyer@chromium.org
Blocking: 457440
Components: Test>Android Infra>Platform>Buildbot
Labels: -Pri-2 Pri-1
After talking to yosin@, this is suspected to be a problem with the bot's config for ICU.
Cc: nick@chromium.org
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?
Cc: -yosin@chromium.org
Owner: yosin@chromium.org
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. 
Project Member

Comment 10 by bugdroid1@chromium.org, 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

Comment 11 by torne@chromium.org, Oct 25 2016

 Issue 657331  has been merged into this issue.
Project Member

Comment 12 by bugdroid1@chromium.org, 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

Comment 13 by torne@chromium.org, Oct 25 2016

Cc: torne@chromium.org

Comment 14 by yosin@chromium.org, Nov 28 2016

Owner: ----
Status: Available (was: Assigned)
Labels: Pri-2
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.
Components: Tests>Disabled
Labels: Test-Disabled

Comment 18 by bokan@chromium.org, Feb 10 2018

Cc: bokan@chromium.org yosin@chromium.org
yosin@, did you ever determine the cause or confirm the theory in #6? 

Comment 19 by no...@chromium.org, Jun 17 2018

Components: -Infra>Platform>Buildbot Infra>Client>Chrome

Sign in to add a comment