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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java

Issue 1445233002: Schedule an early upload of non-JNI Java crashes Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: sync / rebase Created 5 years 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/java/src/org/chromium/chrome/browser/tab/Tab.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java b/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
index 8066da2c07fac086c255d3084dd2e76a86dafaec..eb161b18e959cfd8b2b8c0c7f9b6dd4591b6124c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
@@ -2431,8 +2431,7 @@ public class Tab implements ViewGroup.OnHierarchyChangeListener,
// attempt to do this regardless of whether it was a foreground tab in the event
// that it's a real crash and not just android killing the tab.
Context context = getApplicationContext();
- Intent intent = MinidumpUploadService.createFindAndUploadLastCrashIntent(context);
- context.startService(intent);
+ MinidumpUploadService.tryUploadLastCrashDump(context);
Yaron 2015/12/04 21:29:31 uhm, this isn't what I meant (but it's technically
RecordUserAction.record("MobileBreakpadUploadAttempt");
} catch (SecurityException e) {
// For KitKat and below, there was a framework bug which cause us to not be able to

Powered by Google App Engine
This is Rietveld 408576698