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

Issue 808444 link

Starred by 1 user
Status: Duplicate
Merged: issue 789306
Owner: ----
Closed: Feb 2018
Cc:
perezju@chromium.org
twelling...@chromium.org
ntfschr@chromium.org
█ wnwen@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

testPowerSaveBlockerIsTransferredToFullscreen__multiprocess_mode in webview_instrumentation_test_apk failing on chromium.android/Marshmallow Tablet Tester

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Feb 2 2018 
Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of perezju@google.com

org.chromium.android_webview.test.AwContentsClientFullScreenTest#testPowerSaveBlockerIsTransferredToFullscreen__multiprocess_mode in webview_instrumentation_test_apk failing on chromium.android/Marshmallow Tablet Tester

First failure was:
https://ci.chromium.org/buildbot/chromium.android/Marshmallow%20Tablet%20Tester/11649

The failure appears to be flaky looking at:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=org.chromium.android_webview.test.AwContentsClientFullScreenTest%23testPowerSaveBlockerIsTransferredToFullscreen__multiprocess_mode&testType=webview_instrumentation_test_apk

Comment 2 by perezju@chromium.org, Feb 2 2018

Cc: wnwen@chromium.org twelling...@chromium.org
Error message:

java.lang.AssertionError: Values should be different. Actual: true
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failEquals(Assert.java:185)
	at org.junit.Assert.assertNotEquals(Assert.java:161)
	at org.junit.Assert.assertNotEquals(Assert.java:175)
	at org.chromium.android_webview.test.AwContentsClientFullScreenTest.assertKeepScreenOnActive(AwContentsClientFullScreenTest.java:390)
	at org.chromium.android_webview.test.AwContentsClientFullScreenTest.testPowerSaveBlockerIsTransferredToFullscreen(AwContentsClientFullScreenTest.java:317)
[...]

+wnwen, could your CL be related?

Android: Remove more unused resources
https://chromium-review.googlesource.com/896405

Comment 3 by perezju@chromium.org, Feb 2 2018

Cc: ntfschr@chromium.org
Disabling test:
https://chromium-review.googlesource.com/c/chromium/src/+/899245

Comment 4 by perezju@chromium.org, Feb 2 2018

Components: Mobile>WebView
Labels: -Pri-0 Pri-2

Comment 5 by twelling...@chromium.org, Feb 2 2018 

I doubt Peter's change to remove unused resources is related. That CL just deletes some unused styles/colors/dimensions and removes some lint suppression. Issues related to Peter's change would most likely be failed compilation. The fullscreen_notification_in.xml that was removed is an animation style that is no longer used. I don't believe it ever could have been used by webview because webview doesn't depend on resources in chrome/.
Project Member

Comment 6 by bugdroid1@chromium.org, Feb 2 2018 

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

commit 58e060dd9de6bd597f0111e4789cc6bce26b197a
Author: Juan Antonio Navarro Perez <perezju@google.com>
Date: Fri Feb 02 17:12:37 2018

Disable testPowerSaveBlockerIsTransferredToFullscreen

Test failing on tablet testers.

TBR: ntfschr@chromium.org
Bug:  808444 
Change-Id: I0f18bded5aa2e5204be8f745b5ec1ae16fcc100a
Reviewed-on: https://chromium-review.googlesource.com/899245
Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534080}
[modify] https://crrev.com/58e060dd9de6bd597f0111e4789cc6bce26b197a/android_webview/javatests/src/org/chromium/android_webview/test/AwContentsClientFullScreenTest.java

Comment 7 by ntfschr@chromium.org, Feb 8 2018

Mergedinto: 789306
Status: Duplicate (was: Available)
Probably the same flake as  issue 789306 .

Comment 8 by ntfschr@chromium.org, Aug 23

Labels: WebView-Disabled-Test
►

Sign in to add a comment