Fix test crashes related to AccountManagerHelper.

chrome/android/javatests/src/org/chromium/chrome/browser/precache/PrecacheLauncherTest.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.precache;
 
 import static org.chromium.base.test.util.Restriction.RESTRICTION_TYPE_NON_LOW_END_DEVICE;
 
 import android.content.Context;
 import android.support.test.InstrumentationRegistry;
 import android.support.test.filters.SmallTest;
 
 import com.google.android.gms.gcm.Task;
 
 import org.junit.After;
 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.ContextUtils;
 import org.chromium.base.ThreadUtils;
 import org.chromium.base.test.BaseJUnit4ClassRunner;
 import org.chromium.base.test.util.Feature;
 import org.chromium.base.test.util.Restriction;
 import org.chromium.chrome.browser.preferences.privacy.PrivacyPreferencesManager;
 import org.chromium.chrome.browser.sync.ProfileSyncService;
-import org.chromium.content.browser.test.NativeLibraryTestRule;
+import org.chromium.chrome.browser.test.ChromeBrowserTestRule;
 
 import java.util.EnumSet;
 import java.util.concurrent.Callable;
 
 /**
  * Unit tests for {@link PrecacheLauncher}.
  *
  * setUp/tearDown code was inspired by org.chromium.chrome.browser.sync.ui.PassphraseActivityTest.
  */
 @RunWith(BaseJUnit4ClassRunner.class)
 public class PrecacheLauncherTest {
     @Rule
-    public NativeLibraryTestRule mActivityTestRule = new NativeLibraryTestRule();
+    public final ChromeBrowserTestRule mBrowserTestRule = new ChromeBrowserTestRule();
 
     private StubProfileSyncService mSync;
     private PrecacheLauncherUnderTest mLauncher;
     private MockPrecacheTaskScheduler mPrecacheTaskScheduler;
 
     private class PrecacheLauncherUnderTest extends PrecacheLauncher {
         private boolean mShouldRun = false;
 
         @Override
         protected void onPrecacheCompleted(boolean tryAgainSoon) {}
 
         @Override
         boolean nativeShouldRun() {
             return mShouldRun;
         }
 
-        /**
+        /**R

[Ted C, 2017/06/29 20:20:55]

STUVWXYandZ!

[troyhildebrandt, 2017/06/29 20:43:48]

Congratulations on finding the easter egg...? Done.

          * Modify the return value of nativeShouldRun. This will notify sync state subscribers, as
          * if the user changed their sync preferences.
          */
         void setShouldRun(boolean shouldRun) {
             mShouldRun = shouldRun;
             notifySyncChanged();
         }
     }
 
     private static class StubProfileSyncService extends ProfileSyncService {
@@ skipping 35 matching lines @@
         mLauncher = new PrecacheLauncherUnderTest();
 
         mPrecacheTaskScheduler = new MockPrecacheTaskScheduler();
         PrecacheController.setTaskScheduler(mPrecacheTaskScheduler);
 
         // The target context persists throughout the entire test run, and so leaks state between
         // tests. We reset the is_precaching_enabled pref to false to make the test run consistent,
         // in case another test class has modified this pref.
         PrecacheController.setIsPrecachingEnabled(getTargetContext(), false);
 
-        // ProfileSyncService needs the browser process to be running.
-        mActivityTestRule.loadNativeLibraryAndInitBrowserProcess();
-
         // ProfileSyncService must be initialized on the UI thread. Oddly, even though
         // ThreadUtils.runningOnUiThread() is true here, it's, no, not really, no.
         ThreadUtils.runOnUiThreadBlocking(new Runnable() {
             @Override
             public void run() {
                 // The StubProfileSyncService stubs out isEngineInitialized so we can change that
                 // on the fly.
                 mSync = new StubProfileSyncService();
                 ProfileSyncService.overrideForTests(mSync);
                 // This is currently the default, but let's verify that, lest it ever change and we
@@ skipping 143 matching lines @@
     /** Notify listeners that sync preferences have changed. This is run by setShouldRun. */
     private void notifySyncChanged() {
         ThreadUtils.runOnUiThreadBlocking(new Runnable() {
             @Override
             public void run() {
                 mSync.syncStateChanged();
             }
         });
     }
 }