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

Issue 767624 link

Starred by 0 users

Issue metadata

Status: WontFix
Owner:
OOO until Feb 4th
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 774019



Sign in to add a comment

Investigate StrictMode violation causing flaky tests

Project Member Reported by wnwen@chromium.org, Sep 21 2017

Issue description

 issue 767423  and  issue 767153  are causing flaky tests.

First - whitelist affected codepaths
Second - investigate reason behind flakiness
Third - remove whitelisting if it makes sense

Please link other StrictMode bugs that affect tests here.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 21 2017

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

commit 908f3f7144b2598348d33a668d494f85070bba41
Author: Peter Wen <wnwen@chromium.org>
Date: Thu Sep 21 22:54:55 2017

Android: Patch flaky tests due to strict mode

Android system is crashing due to StrictMode but we need to know the
underlying cause in order to understand why it is flaking. Disabling
StrictMode just for those code paths as the flakiness is unreliable and
hard to reproduce.

BUG= 767423 , 767153 , 767624 

Change-Id: I0ddcabd876bab99841487742f039cea920ea433d
Reviewed-on: https://chromium-review.googlesource.com/677226
Commit-Queue: Peter Wen <wnwen@chromium.org>
Reviewed-by: ccameron chromium <ccameron@chromium.org>
Reviewed-by: Eric Stevenson <estevenson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#503595}
[modify] https://crrev.com/908f3f7144b2598348d33a668d494f85070bba41/ui/android/java/src/org/chromium/ui/base/WindowAndroid.java

The cause of the StrictMode violation seems to be a Slog.wtf call in ActivityManagerService (https://android.googlesource.com/platform/frameworks/base/+/android-4.4.4_r2.0.1/services/java/com/android/server/am/ActiveServices.java#2081):

  // Sanity check: if the service listed for the app is not one
  // we actually are maintaining, just let it drop.
  if (smap.mServicesByName.get(sr.name) != sr) {
      ServiceRecord cur = smap.mServicesByName.get(sr.name);
      Slog.wtf(TAG, "Service " + sr + " in process " + app
              + " not same as in map: " + cur);
      continue;
  }


Comment 3 by dgn@chromium.org, Oct 17 2017

Blocking: 774019

Comment 4 by wnwen@chromium.org, Nov 16 2017

Status: WontFix (was: Assigned)
Slog.wtf would have caused a crash anyways, so I think this is a flake in Android that we can't really fix (even if we turn off StrictMode completely which I think loses much benefit). Hopefully we will migrate off Android 4 for testing in the coming years.

See Log.wtf in android code.

Sign in to add a comment