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

Side by Side 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: Split up everything to the idle handler. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser; 5 package org.chromium.chrome.browser;
6 6
7 import android.app.Activity; 7 import android.app.Activity;
8 import android.app.ActivityManager; 8 import android.app.ActivityManager;
9 import android.content.Context; 9 import android.content.Context;
10 import android.content.Intent; 10 import android.content.Intent;
(...skipping 936 matching lines...) Expand 10 before | Expand all | Expand 10 after
947 mHandler.post(new Runnable() { 947 mHandler.post(new Runnable() {
948 @Override 948 @Override
949 public void run() { 949 public void run() {
950 startActivityForResult(freIntent, FIRST_RUN_EXPERIENCE_RESULT); 950 startActivityForResult(freIntent, FIRST_RUN_EXPERIENCE_RESULT);
951 } 951 }
952 }); 952 });
953 } 953 }
954 954
955 @Override 955 @Override
956 protected void onDeferredStartup() { 956 protected void onDeferredStartup() {
957 try { 957 DeferredStartupHandler.getInstance().addDeferredTask(new Runnable() {
958 TraceEvent.begin("ChromeTabbedActivity.onDeferredStartup"); 958 @Override
959 super.onDeferredStartup(); 959 public void run() {
960 ActivityManager am =
961 (ActivityManager) getSystemService(Context.ACTIVITY_SERV ICE);
962 RecordHistogram.recordSparseSlowlyHistogram(
963 "MemoryAndroid.DeviceMemoryClass", am.getMemoryClass());
960 964
961 ActivityManager am = (ActivityManager) getSystemService(Context.ACTI VITY_SERVICE); 965 AutocompleteController.nativePrefetchZeroSuggestResults();
962 RecordHistogram.recordSparseSlowlyHistogram( 966 }
963 "MemoryAndroid.DeviceMemoryClass", am.getMemoryClass()); 967 });
964 968 super.onDeferredStartup();
Yaron 2016/07/06 14:27:49 the ordering of this chnaged?
Peter Wen 2016/07/06 15:21:36 Yes, do you think that will cause a problem? This
965 AutocompleteController.nativePrefetchZeroSuggestResults();
966 } finally {
967 TraceEvent.end("ChromeTabbedActivity.onDeferredStartup");
968 }
969 } 969 }
970 970
971 private void createTabModelSelectorImpl(Bundle savedInstanceState) { 971 private void createTabModelSelectorImpl(Bundle savedInstanceState) {
972 // We determine the model as soon as possible so every systems get initi alized coherently. 972 // We determine the model as soon as possible so every systems get initi alized coherently.
973 boolean startIncognito = savedInstanceState != null 973 boolean startIncognito = savedInstanceState != null
974 && savedInstanceState.getBoolean("is_incognito_selected", false) ; 974 && savedInstanceState.getBoolean("is_incognito_selected", false) ;
975 int index = savedInstanceState != null ? savedInstanceState.getInt(WINDO W_INDEX, 0) : 0; 975 int index = savedInstanceState != null ? savedInstanceState.getInt(WINDO W_INDEX, 0) : 0;
976 mTabModelSelectorImpl = (TabModelSelectorImpl) 976 mTabModelSelectorImpl = (TabModelSelectorImpl)
977 TabWindowManager.getInstance().requestSelector(this, getWindowAn droid(), index); 977 TabWindowManager.getInstance().requestSelector(this, getWindowAn droid(), index);
978 if (mTabModelSelectorImpl == null) { 978 if (mTabModelSelectorImpl == null) {
(...skipping 446 matching lines...) Expand 10 before | Expand all | Expand 10 after
1425 if (getActivityTab() != null) { 1425 if (getActivityTab() != null) {
1426 setStatusBarColor(getActivityTab(), getActivityTab().getThemeColor() ); 1426 setStatusBarColor(getActivityTab(), getActivityTab().getThemeColor() );
1427 } 1427 }
1428 } 1428 }
1429 1429
1430 @Override 1430 @Override
1431 protected void setStatusBarColor(Tab tab, int color) { 1431 protected void setStatusBarColor(Tab tab, int color) {
1432 super.setStatusBarColor(tab, isInOverviewMode() ? Color.BLACK : color); 1432 super.setStatusBarColor(tab, isInOverviewMode() ? Color.BLACK : color);
1433 } 1433 }
1434 } 1434 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698