Move initialization tasks from DeferredStartupHandler to ProcessInitializationHandler |
||
Issue descriptionFollowing email conversation with Ted. Right now startup tasks are added from both ProcessInitializationHandler.handleDeferredStartupTasksInitialization and DeferredStartupHandler.initDeferredStartupForApp. Splitting responsibilities seems cleaner because then activities and application level tasks can be added without handler class getting so bloated.
,
May 2 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5367fc237daf3ebf4fae4153fe3f8ccc60910f51 commit 5367fc237daf3ebf4fae4153fe3f8ccc60910f51 Author: bsazonov <bsazonov@chromium.org> Date: Tue May 02 16:53:28 2017 Move second part of initialization tasks to ProcessInitializationHandler This CL continues initialization task movement started by http://crrev.com/2853513002. BUG= 716454 Review-Url: https://codereview.chromium.org/2852193003 Cr-Commit-Position: refs/heads/master@{#468681} [modify] https://crrev.com/5367fc237daf3ebf4fae4153fe3f8ccc60910f51/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java [modify] https://crrev.com/5367fc237daf3ebf4fae4153fe3f8ccc60910f51/chrome/android/java/src/org/chromium/chrome/browser/init/ProcessInitializationHandler.java
,
May 5 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5b08619bdacb059f9523b14a5ed84dddf33e9275 commit 5b08619bdacb059f9523b14a5ed84dddf33e9275 Author: bsazonov <bsazonov@chromium.org> Date: Fri May 05 07:47:50 2017 Move final part of initialization tasks to ProcessInitializationHandler This CL finishes initialization task movement started by http://crrev.com/2853513002 and http://crrev.com/2852193003. BUG= 716454 Review-Url: https://codereview.chromium.org/2858893002 Cr-Commit-Position: refs/heads/master@{#469613} [modify] https://crrev.com/5b08619bdacb059f9523b14a5ed84dddf33e9275/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java [modify] https://crrev.com/5b08619bdacb059f9523b14a5ed84dddf33e9275/chrome/android/java/src/org/chromium/chrome/browser/DeferredStartupHandler.java [modify] https://crrev.com/5b08619bdacb059f9523b14a5ed84dddf33e9275/chrome/android/java/src/org/chromium/chrome/browser/init/ProcessInitializationHandler.java
,
May 5 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, May 2 2017