Reland of Separate deferred startup into tasks.

Reland of Separate deferred startup into tasks.

Original: https://codereview.chromium.org/2121863002/

Fixes:
- Guarantee ChromeApplication#onStartWithNative has been called by
  custom tabs before posting deferred startup.

BUG=614452
TBR=dtrainor@chromium.org,agrieve@chromium.org,asvitkine@chromium.org,pasko@chromium.org

Committed: https://crrev.com/bcdc694e5fb4cf003124cf5b940e4807322a184e
Cr-Commit-Position: refs/heads/master@{#410342}

[Peter Wen, 2016-08-03 14:07:14]

Description was changed from

==========
Reland of Separate deferred startup into tasks.

Original: https://codereview.chromium.org/2121863002/

Fixes:
- Remove custom tabs' early call to post deferred startup.

BUG=614452
TBR=dtrainor@chromium.org,agrieve@chromium.org,asvitkine@chromium.org,pasko@c...
==========

to

==========
Reland of Separate deferred startup into tasks.

Original: https://codereview.chromium.org/2121863002/

Fixes:
- Guarantee ChromeApplication#onStartWithNative has been called by
  custom tabs before posting deferred startup.

BUG=614452
TBR=dtrainor@chromium.org,agrieve@chromium.org,asvitkine@chromium.org,pasko@c...
==========

[Peter Wen, 2016-08-03 14:08:42]

wnwen@chromium.org changed reviewers:
+ yusufo@chromium.org

[Peter Wen, 2016-08-03 14:08:43]

Hi Yusuf,

Can you take a look at the fix for custom tabs test failure?
postDeferredStartupIfNeeded was being called before ChromeApplication has
initialized UmaUtils foreground time. I've moved that call to onStartWithNative
instead.

Thanks,

Peter

[Peter Wen, 2016-08-03 14:08:48]

The CQ bit was checked by wnwen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-03 14:09:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Peter Wen, 2016-08-03 14:09:34]

cc: yfriedman@ and tedchoc@ for original CL and revert.

[Alexei Svitkine (slow), 2016-08-03 14:57:07]

lgtm

[commit-bot: I haz the power, 2016-08-03 15:19:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-03 15:19:40]

Dry run: This issue passed the CQ dry run.

[Yusuf, 2016-08-03 17:22:22]

lgtm

https://codereview.chromium.org/2207933002/diff/20001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabActivity.java
(right):

https://codereview.chromium.org/2207933002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabActivity.java:482:
if (mHasCreatedTabEarly && !mMainTab.isLoading()) {
I think this can fit in one line?

[David Trainor- moved to gerrit, 2016-08-04 16:06:22]

rubber stamp lgtm on chrome/android from previous CL.

[Peter Wen, 2016-08-08 12:45:49]

The CQ bit was checked by wnwen@chromium.org

[Peter Wen, 2016-08-08 12:45:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org, asvitkine@chromium.org, yusufo@chromium.org
Link to the patchset: https://codereview.chromium.org/2207933002/#ps40001
(title: "Fix one-line.")

[commit-bot: I haz the power, 2016-08-08 12:45:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Peter Wen, 2016-08-08 12:46:01]

Thanks for the reviews!

https://codereview.chromium.org/2207933002/diff/20001/chrome/android/java/src...
File
chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabActivity.java
(right):

https://codereview.chromium.org/2207933002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabActivity.java:482:
if (mHasCreatedTabEarly && !mMainTab.isLoading()) {
On 2016/08/03 17:22:22, Yusuf wrote:
> I think this can fit in one line?

Done.

[commit-bot: I haz the power, 2016-08-08 13:52:22]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-08 13:53:59]

Description was changed from

==========
Reland of Separate deferred startup into tasks.

Original: https://codereview.chromium.org/2121863002/

Fixes:
- Guarantee ChromeApplication#onStartWithNative has been called by
  custom tabs before posting deferred startup.

BUG=614452
TBR=dtrainor@chromium.org,agrieve@chromium.org,asvitkine@chromium.org,pasko@c...
==========

to

==========
Reland of Separate deferred startup into tasks.

Original: https://codereview.chromium.org/2121863002/

Fixes:
- Guarantee ChromeApplication#onStartWithNative has been called by
  custom tabs before posting deferred startup.

BUG=614452
TBR=dtrainor@chromium.org,agrieve@chromium.org,asvitkine@chromium.org,pasko@c...

Committed: https://crrev.com/bcdc694e5fb4cf003124cf5b940e4807322a184e
Cr-Commit-Position: refs/heads/master@{#410342}
==========

[commit-bot: I haz the power, 2016-08-08 13:54:00]

Patchset 3 (id:??) landed as
https://crrev.com/bcdc694e5fb4cf003124cf5b940e4807322a184e
Cr-Commit-Position: refs/heads/master@{#410342}