Convert WebappActivityTestBase and direct children to JUnit4.

chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebappSplashScreenHomescreenIconTest.java

 // Copyright 2015 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.chrome.browser.webapps;
 
 import android.content.Intent;
 import android.graphics.Bitmap;
 import android.graphics.drawable.BitmapDrawable;
 import android.support.test.filters.SmallTest;
 import android.view.ViewGroup;
 import android.widget.ImageView;
 
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
 import org.chromium.base.metrics.RecordHistogram;
+import org.chromium.base.test.util.CommandLineFlags;
 import org.chromium.base.test.util.Feature;
 import org.chromium.base.test.util.RetryOnFailure;
 import org.chromium.chrome.R;
+import org.chromium.chrome.browser.ChromeSwitches;
 import org.chromium.chrome.browser.ShortcutHelper;
 import org.chromium.chrome.browser.metrics.WebappUma;
+import org.chromium.chrome.test.ChromeActivityTestRule;
+import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
 
 /**
  * Tests for splash screens with EXTRA_ICON specified in the Intent.
  */
-public class WebappSplashScreenHomescreenIconTest extends WebappActivityTestBase {
-    @Override
-    protected void setUp() throws Exception {
-        super.setUp();
-        startWebappActivity();
+@RunWith(ChromeJUnit4ClassRunner.class)
+@CommandLineFlags.Add({ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE,
+        ChromeActivityTestRule.DISABLE_NETWORK_PREDICTION_FLAG})
+public class WebappSplashScreenHomescreenIconTest {
+    @Rule
+    public final WebappActivityTestRule mActivityTestRule = new WebappActivityTestRule();
+
+    @Before
+    public void setUp() throws Exception {
+        mActivityTestRule.startWebappActivity(createIntent());
     }
 
-    @Override
-    protected Intent createIntent() {
-        Intent intent = super.createIntent();
-        intent.putExtra(ShortcutHelper.EXTRA_ICON, TEST_ICON);
+    private Intent createIntent() {

[the real yoland, 2017/05/04 19:04:43]

I would recommend just change the createIntent() method in
WebappActivityTestRule to createIntent(String name, String value) and not create
a new method here

[piotrs, 2017/05/05 01:15:53]

I got around it just inlining createIntent() with putExtra(), shorter and nicer.
creating createIntent(String name, String value) would be fine as well, but one
tests requires to additional extras, so it's less flexible.

+        Intent intent = mActivityTestRule.createIntent();
+        intent.putExtra(ShortcutHelper.EXTRA_ICON, WebappActivityTestRule.TEST_ICON);
         return intent;
     }
 
+    @Test
     @SmallTest
     @Feature({"Webapps"})
     public void testShowFallbackIcon() {
-        ViewGroup splashScreen = waitUntilSplashScreenAppears();
+        ViewGroup splashScreen = mActivityTestRule.waitUntilSplashScreenAppears();
         ImageView splashImage = (ImageView) splashScreen.findViewById(
                 R.id.webapp_splash_screen_icon);
         BitmapDrawable drawable = (BitmapDrawable) splashImage.getDrawable();
 
-        assertEquals(192, drawable.getBitmap().getWidth());
-        assertEquals(192, drawable.getBitmap().getHeight());
+        Assert.assertEquals(192, drawable.getBitmap().getWidth());
+        Assert.assertEquals(192, drawable.getBitmap().getHeight());
     }
 
+    @Test
     @SmallTest
     @Feature({"Webapps"})
     @RetryOnFailure
     public void testUmaFallbackIcon() {
-        assertEquals(1, RecordHistogram.getHistogramValueCountForTesting(
-                WebappUma.HISTOGRAM_SPLASHSCREEN_ICON_TYPE,
-                WebappUma.SPLASHSCREEN_ICON_TYPE_FALLBACK));
+        Assert.assertEquals(1,
+                RecordHistogram.getHistogramValueCountForTesting(
+                        WebappUma.HISTOGRAM_SPLASHSCREEN_ICON_TYPE,
+                        WebappUma.SPLASHSCREEN_ICON_TYPE_FALLBACK));
 
-        Bitmap icon = ShortcutHelper.decodeBitmapFromString(TEST_ICON);
+        Bitmap icon = ShortcutHelper.decodeBitmapFromString(WebappActivityTestRule.TEST_ICON);
         int sizeInDp = Math.round((float) icon.getWidth()
-                / getActivity().getResources().getDisplayMetrics().density);
-        assertEquals(1, RecordHistogram.getHistogramValueCountForTesting(
-                WebappUma.HISTOGRAM_SPLASHSCREEN_ICON_SIZE, sizeInDp));
+                / mActivityTestRule.getActivity().getResources().getDisplayMetrics().density);
+        Assert.assertEquals(1,
+                RecordHistogram.getHistogramValueCountForTesting(
+                        WebappUma.HISTOGRAM_SPLASHSCREEN_ICON_SIZE, sizeInDp));
     }
 }