Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(24)

Issue 2954183002: Attempt to prevent screenshot failures from preventing original failure stack. (Closed)

Created:
3 years, 6 months ago by Ted C
Modified:
3 years, 6 months ago
Reviewers:
jbudorick
CC:
chromium-reviews, danakj+watch_chromium.org, vmpstr+watch_chromium.org, mikecase (-- gone --)
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Attempt to prevent screenshot failures from preventing original failure stack. Saw an exception like this on the bots: java.lang.RuntimeException: Error while connecting UiAutomation at android.app.UiAutomation.connect(UiAutomation.java:203) at android.app.Instrumentation.getUiAutomation(Instrumentation.java:1680) at android.support.test.uiautomator.j.<init>(UiDevice.java:90) at android.support.test.uiautomator.j.a(UiDevice.java:249) at org.chromium.base.test.ScreenshotOnFailureStatement.evaluate(ScreenshotOnFailureStatement.java:1055) at android.support.test.internal.a.b.run(UiThreadStatement.java:44) at android.app.Instrumentation$SyncRunnable.run(Instrumentation.java:1719) at android.os.Handler.handleCallback(Handler.java:733) at android.os.Handler.dispatchMessage(Handler.java:95) at android.os.Looper.loop(Looper.java:136) at android.app.ActivityThread.main(ActivityThread.java:5001) at java.lang.reflect.Method.invokeNative(Native Method) at java.lang.reflect.Method.invoke(Method.java:515) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:785) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:601) at dalvik.system.NativeStart.main(Native Method) Which prevented me from seeing the original failure. BUG=736380 Review-Url: https://codereview.chromium.org/2954183002 Cr-Commit-Position: refs/heads/master@{#482004} Committed: https://chromium.googlesource.com/chromium/src/+/27a522ddf6ece1cccf741e309e035bf0d461657a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -1 line) Patch
M base/test/android/javatests/src/org/chromium/base/test/ScreenshotOnFailureStatement.java View 1 chunk +8 lines, -1 line 0 comments Download

Messages

Total messages: 9 (4 generated)
Ted C
PTAL
3 years, 6 months ago (2017-06-23 19:06:19 UTC) #2
Ted C
On 2017/06/23 19:06:19, Ted C wrote: > PTAL Failing build: https://build.chromium.org/p/chromium.linux/builders/Android%20Tests/builds/43039
3 years, 6 months ago (2017-06-23 19:06:41 UTC) #3
jbudorick
lgtm +mikecase fyi
3 years, 6 months ago (2017-06-23 19:11:27 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2954183002/1
3 years, 6 months ago (2017-06-23 19:22:42 UTC) #6
commit-bot: I haz the power
3 years, 6 months ago (2017-06-23 20:22:44 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/27a522ddf6ece1cccf741e309e03...

Powered by Google App Engine
This is Rietveld 408576698