Fixed a crash in blimp toolbar

Fixed a crash in blimp toolbar

On a slow device, sometimes onNewIntent is triggered even before the loading of libraries is completed.
In that case blimp currently crashes while trying to load an URL since toolbar is not yet initialized. Fixed the crash by adding a null check.

BUG=601226
Committed: https://crrev.com/daa11718e228d5eb00640e2716016b24d24187d0
Cr-Commit-Position: refs/heads/master@{#387045}

[shaktisahu, 2016-04-07 23:32:58]

Description was changed from

==========
Fixed a crash in toolbar

BUG=601226
==========

to

==========
Fixed a crash in toolbar

BUG=601226
==========

[shaktisahu, 2016-04-07 23:32:58]

shaktisahu@chromium.org changed reviewers:
+ nyquist@chromium.org

[shaktisahu, 2016-04-08 21:21:18]

Description was changed from

==========
Fixed a crash in toolbar

BUG=601226
==========

to

==========
Fixed a crash in blimp toolbar

On a slow device, sometimes onNewIntent is triggered even before the loading of
libraries is completed.
In that case blimp currently crashes while trying to load an URL since toolbar
is not yet initialized. Fixed the crash by adding a null check.

BUG=601226
==========

[nyquist, 2016-04-12 18:54:29]

So; this fixes the crash; but we lose the Intent that was being opened. Should
you maybe add a TODO to ensure that we load the correct URL when the toolbar has
been loaded?

[nyquist, 2016-04-13 17:50:05]

The CQ bit was checked by nyquist@chromium.org

[nyquist, 2016-04-13 17:50:05]

lgtm

[commit-bot: I haz the power, 2016-04-13 17:50:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1875523002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1875523002/20001

[commit-bot: I haz the power, 2016-04-13 18:39:44]

Description was changed from

==========
Fixed a crash in blimp toolbar

On a slow device, sometimes onNewIntent is triggered even before the loading of
libraries is completed.
In that case blimp currently crashes while trying to load an URL since toolbar
is not yet initialized. Fixed the crash by adding a null check.

BUG=601226
==========

to

==========
Fixed a crash in blimp toolbar

On a slow device, sometimes onNewIntent is triggered even before the loading of
libraries is completed.
In that case blimp currently crashes while trying to load an URL since toolbar
is not yet initialized. Fixed the crash by adding a null check.

BUG=601226
==========

[commit-bot: I haz the power, 2016-04-13 18:39:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-13 18:41:55]

Description was changed from

==========
Fixed a crash in blimp toolbar

On a slow device, sometimes onNewIntent is triggered even before the loading of
libraries is completed.
In that case blimp currently crashes while trying to load an URL since toolbar
is not yet initialized. Fixed the crash by adding a null check.

BUG=601226
==========

to

==========
Fixed a crash in blimp toolbar

On a slow device, sometimes onNewIntent is triggered even before the loading of
libraries is completed.
In that case blimp currently crashes while trying to load an URL since toolbar
is not yet initialized. Fixed the crash by adding a null check.

BUG=601226

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

[commit-bot: I haz the power, 2016-04-13 18:41:57]

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