| Index: chrome/android/javatests/src/org/chromium/chrome/browser/crash/MinidumpUploadServiceTest.java
|
| diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/crash/MinidumpUploadServiceTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/crash/MinidumpUploadServiceTest.java
|
| index 18db8116b75a4e81a7af13d5aae6f1da46fc9250..ef017e7f9da037f524bff7f56a48a6f764a2c908 100644
|
| --- a/chrome/android/javatests/src/org/chromium/chrome/browser/crash/MinidumpUploadServiceTest.java
|
| +++ b/chrome/android/javatests/src/org/chromium/chrome/browser/crash/MinidumpUploadServiceTest.java
|
| @@ -4,24 +4,21 @@
|
|
|
| package org.chromium.chrome.browser.crash;
|
|
|
| -import static org.chromium.components.minidump_uploader.util.MinidumpUploadDelegate.BROWSER;
|
| -import static org.chromium.components.minidump_uploader.util.MinidumpUploadDelegate.GPU;
|
| -import static org.chromium.components.minidump_uploader.util.MinidumpUploadDelegate.OTHER;
|
| -import static org.chromium.components.minidump_uploader.util.MinidumpUploadDelegate.RENDERER;
|
| +import static org.chromium.chrome.browser.crash.MinidumpUploadService.BROWSER;
|
| +import static org.chromium.chrome.browser.crash.MinidumpUploadService.GPU;
|
| +import static org.chromium.chrome.browser.crash.MinidumpUploadService.OTHER;
|
| +import static org.chromium.chrome.browser.crash.MinidumpUploadService.RENDERER;
|
|
|
| import android.content.ComponentName;
|
| import android.content.Context;
|
| import android.content.Intent;
|
| import android.os.Handler;
|
| import android.os.HandlerThread;
|
| -import android.test.MoreAsserts;
|
| import android.test.suitebuilder.annotation.SmallTest;
|
|
|
| import org.chromium.base.annotations.SuppressFBWarnings;
|
| import org.chromium.base.test.util.AdvancedMockContext;
|
| import org.chromium.base.test.util.Feature;
|
| -import org.chromium.components.minidump_uploader.util.CrashReportingPermissionManager;
|
| -import org.chromium.components.minidump_uploader.util.MinidumpUploadDelegate;
|
| import org.chromium.content.browser.test.util.Criteria;
|
| import org.chromium.content.browser.test.util.CriteriaHelper;
|
| import org.chromium.net.NetworkChangeNotifier;
|
| @@ -40,13 +37,6 @@
|
| private static final int MAX_TIMEOUT_MS = 20000;
|
| private static final String BOUNDARY = "TESTBOUNDARY";
|
|
|
| - @Override
|
| - public void setUp() throws Exception {
|
| - super.setUp();
|
| -
|
| - MinidumpUploadService.setUploadDelegate(new TestMinidumpUploadDelegate());
|
| - }
|
| -
|
| @SmallTest
|
| @Feature({"Android-AppBase"})
|
| public void testFindAndUploadLastCrash() throws IOException {
|
| @@ -83,32 +73,6 @@
|
| // Verify.
|
| assertTrue("Minidump file should exist", minidumpFile.isFile());
|
| assertEquals("Should have called startService() once", 1, numServiceStarts.intValue());
|
| - }
|
| -
|
| - private static class TestMinidumpUploadDelegate implements MinidumpUploadDelegate {
|
| - @Override
|
| - public void onSuccessfulUpload(Context context, @ProcessType String crashType) {}
|
| -
|
| - @Override
|
| - public void onMaxedOutUploadFailures(Context context, @ProcessType String crashType) {
|
| - MoreAsserts.assertNotEqual(crashType, MinidumpUploadDelegate.OTHER);
|
| - }
|
| -
|
| - @Override
|
| - public CrashReportingPermissionManager getCrashReportingPermissionManager() {
|
| - return new MinidumpUploadCallableTest.MockCrashReportingPermissionManager() {
|
| - {
|
| - // None of the tests in this class depend on the values in the permission
|
| - // manager - so set the values to false.
|
| - mIsInSample = false;
|
| - mIsNetworkAvailable = false;
|
| - mIsPermitted = false;
|
| - mIsUserPermitted = false;
|
| - mIsCommandLineDisabled = false;
|
| - mIsEnabledForTests = false;
|
| - }
|
| - };
|
| - }
|
| }
|
|
|
| private static class TestMinidumpUploadService extends MinidumpUploadService {
|
| @@ -349,7 +313,7 @@
|
| File minidumpFile = new File(mCrashDir, "chromium_renderer-111.dmp1");
|
| minidumpFile.createNewFile();
|
| File logfile = new File(mCrashDir, CrashFileManager.CRASH_DUMP_LOGFILE);
|
| - setUpMinidumpFile(minidumpFile, BOUNDARY, "renderer");
|
| + setUpMinidumpFile(minidumpFile, BOUNDARY);
|
|
|
| // Run test.
|
| Intent uploadIntent =
|
|
|