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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/crash/LogcatExtractionRunnableTest.java

Issue 2737263006: [Android Crash Reporting] Allow uploading minidumps via the JobScheduler (Closed)
Patch Set: Use shared prefs Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/javatests/src/org/chromium/chrome/browser/crash/LogcatExtractionRunnableTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/crash/LogcatExtractionRunnableTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/crash/LogcatExtractionRunnableTest.java
index cd851953dff3bf5634e50b2c5a3716357c21a30d..096ddb93926e56bf8385312b5b86cfbfe7fb3f4c 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/crash/LogcatExtractionRunnableTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/crash/LogcatExtractionRunnableTest.java
@@ -29,6 +29,38 @@ import java.util.concurrent.atomic.AtomicInteger;
public class LogcatExtractionRunnableTest extends CrashTestCase {
private File mCrashDir;
+ private class TestJobScheduler extends JobScheduler {
+ private final JobScheduler mRealScheduler;
+
+ TestJobScheduler(JobScheduler realScheduler) {
+ mRealScheduler = realScheduler;
+ }
+
+ @Override
+ public void cancel(int jobId) {
+ mRealScheduler.cancel(jobId);
+ }
+ @Override
+ public void cancelAll() {
+ mRealScheduler.cancelAll();
+ }
+ @Override
+ public List<JobInfo> getAllPendingJobs() {
+ mRealScheduler.getAllPendingJobs();
+ }
+ @Override
+ public JobInfo getPendingJob() {
+ mRealScheduler.getPendingJob();
+ }
+ @Override
+ public int schedule(JobInfo job) {
+ assertEquals(MinidumpUploadJobService.MINIDUMP_UPLOADING_JOB_ID, job.getId());
+ assertEquals(ChromeMinidumpUploadJobService.class.getName(),
+ job.getService().getClassName());
+ return mRealScheduler.schedule(uploadJob);
+ }
+ };
+
@Override
protected void setUp() throws Exception {
super.setUp();
@@ -70,6 +102,15 @@ public class LogcatExtractionRunnableTest extends CrashTestCase {
intent.getStringExtra(MinidumpUploadService.FILE_TO_UPLOAD_KEY));
return super.startService(intent);
}
+
+ @Override
+ public getSystemService(String name) {
+ if (Context.JOB_SCHEDULER_SERVICE.equals(name)) {
+ return new TestJobScheduler(super.getSystemService(name));
+ }
+
+ return super.getSystemService(name);
+ }
};
// Extract the logcat!

Powered by Google App Engine
This is Rietveld 408576698