Move screenshot capture to Java-side.

base/test/android/javatests/src/org/chromium/base/test/BaseJUnit4ClassRunner.java

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.base.test;
 
 import android.content.Context;
 import android.support.test.InstrumentationRegistry;
 import android.support.test.internal.runner.junit4.AndroidJUnit4ClassRunner;
 import android.support.test.internal.util.AndroidRunnerParams;
 
 import org.junit.runner.notification.RunNotifier;
 import org.junit.runners.model.FrameworkMethod;
 import org.junit.runners.model.InitializationError;
+import org.junit.runners.model.Statement;
 
 import org.chromium.base.CollectionUtil;
 import org.chromium.base.test.BaseTestResult.PreTestHook;
 import org.chromium.base.test.util.DisableIfSkipCheck;
 import org.chromium.base.test.util.MinAndroidSdkLevelSkipCheck;
 import org.chromium.base.test.util.RestrictionSkipCheck;
 import org.chromium.base.test.util.SkipCheck;
 
 import java.lang.reflect.Method;
 import java.util.ArrayList;
@@ skipping 124 matching lines @@
      * Loop through all the {@code SkipCheck}s to confirm whether a test should be ignored
      */
     private boolean shouldSkip(FrameworkMethod method) {
         for (SkipCheck s : mSkipChecks) {
             if (s.shouldSkip(method)) {
                 return true;
             }
         }
         return false;
     }
+
+    @Override
+    protected Statement withAfters(FrameworkMethod method, Object test, Statement base) {
+        base = new ScreenshotOnFailureStatement(base);
+        return super.withAfters(method, test, base);

[jbudorick, 2017/05/05 01:21:58]

nit:

  return super.withAfters(method, test, new ScreenshotOnFailureStatement(base));

?

[mikecase (-- gone --), 2017/05/05 18:09:19]

Done!

+    }
 }