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

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

Issue 2121863002: Separate deferred startup into tasks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix per review. Created 4 years, 5 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/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java
index bdbc679add1dc3b87d37d048cae98dd2c27f57ac..03e89ed452f5b3b023cd35f66604ceeded95f243 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java
@@ -954,18 +954,18 @@ public class ChromeTabbedActivity extends ChromeActivity implements OverviewMode
@Override
protected void onDeferredStartup() {
- try {
- TraceEvent.begin("ChromeTabbedActivity.onDeferredStartup");
- super.onDeferredStartup();
-
- ActivityManager am = (ActivityManager) getSystemService(Context.ACTIVITY_SERVICE);
- RecordHistogram.recordSparseSlowlyHistogram(
- "MemoryAndroid.DeviceMemoryClass", am.getMemoryClass());
+ DeferredStartupHandler.getInstance().addDeferredTask(new Runnable() {
+ @Override
+ public void run() {
+ ActivityManager am =
+ (ActivityManager) getSystemService(Context.ACTIVITY_SERVICE);
+ RecordHistogram.recordSparseSlowlyHistogram(
+ "MemoryAndroid.DeviceMemoryClass", am.getMemoryClass());
- AutocompleteController.nativePrefetchZeroSuggestResults();
- } finally {
- TraceEvent.end("ChromeTabbedActivity.onDeferredStartup");
- }
+ AutocompleteController.nativePrefetchZeroSuggestResults();
+ }
+ });
+ super.onDeferredStartup();
}
private void createTabModelSelectorImpl(Bundle savedInstanceState) {

Powered by Google App Engine
This is Rietveld 408576698