chromeos: Try to avoid shutdown crash in BootTimesRecorder.

chromeos: Try to avoid shutdown crash in BootTimesRecorder.

Call BrowserThread::IsMessageLoopValid() (which bails out if
the BrowserThreadGlobals singleton is uninitialized) before
calling BrowserThread::CurrentlyOn() (which can apparently
deref a null pointer if base::ThreadTaskRunnerHandle hasn't
been initialized).

BUG=609008
Committed: https://crrev.com/9972cd888b194adc1fc6de8bc6e53945846cb57e
Cr-Commit-Position: refs/heads/master@{#410437}

[Daniel Erat, 2016-08-01 16:02:48]

Description was changed from

==========
chromeos: Try to avoid shutdown crash in BootTimesRecorder.

Call BrowserThread::IsMessageLoopValid() (which bails out if
the BrowserThreadGlobals singleton is uninitialized) before
calling BrowserThread::CurrentlyOn() (which crashes).

BUG=609008
==========

to

==========
chromeos: Try to avoid shutdown crash in BootTimesRecorder.

Call BrowserThread::IsMessageLoopValid() (which bails out if
the BrowserThreadGlobals singleton is uninitialized) before
calling BrowserThread::CurrentlyOn() (which can apparently
deref a null pointer if base::ThreadTaskRunnerHandle hasn't
been initialized).

BUG=609008
==========

[Sami, 2016-08-01 18:10:04]

skyostil@chromium.org changed reviewers:
+ skyostil@chromium.org

[Sami, 2016-08-01 18:10:05]

Non-owner lgtm.

[Daniel Erat, 2016-08-08 19:11:47]

derat@chromium.org changed reviewers:
+ achuith@chromium.org

[achuithb, 2016-08-08 19:19:22]

lgtm

[Daniel Erat, 2016-08-08 19:19:48]

The CQ bit was checked by derat@chromium.org

[commit-bot: I haz the power, 2016-08-08 19:20:32]

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

[commit-bot: I haz the power, 2016-08-08 20:29:11]

Description was changed from

==========
chromeos: Try to avoid shutdown crash in BootTimesRecorder.

Call BrowserThread::IsMessageLoopValid() (which bails out if
the BrowserThreadGlobals singleton is uninitialized) before
calling BrowserThread::CurrentlyOn() (which can apparently
deref a null pointer if base::ThreadTaskRunnerHandle hasn't
been initialized).

BUG=609008
==========

to

==========
chromeos: Try to avoid shutdown crash in BootTimesRecorder.

Call BrowserThread::IsMessageLoopValid() (which bails out if
the BrowserThreadGlobals singleton is uninitialized) before
calling BrowserThread::CurrentlyOn() (which can apparently
deref a null pointer if base::ThreadTaskRunnerHandle hasn't
been initialized).

BUG=609008
==========

[commit-bot: I haz the power, 2016-08-08 20:29:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-08 20:31:19]

Description was changed from

==========
chromeos: Try to avoid shutdown crash in BootTimesRecorder.

Call BrowserThread::IsMessageLoopValid() (which bails out if
the BrowserThreadGlobals singleton is uninitialized) before
calling BrowserThread::CurrentlyOn() (which can apparently
deref a null pointer if base::ThreadTaskRunnerHandle hasn't
been initialized).

BUG=609008
==========

to

==========
chromeos: Try to avoid shutdown crash in BootTimesRecorder.

Call BrowserThread::IsMessageLoopValid() (which bails out if
the BrowserThreadGlobals singleton is uninitialized) before
calling BrowserThread::CurrentlyOn() (which can apparently
deref a null pointer if base::ThreadTaskRunnerHandle hasn't
been initialized).

BUG=609008

Committed: https://crrev.com/9972cd888b194adc1fc6de8bc6e53945846cb57e
Cr-Commit-Position: refs/heads/master@{#410437}
==========

[commit-bot: I haz the power, 2016-08-08 20:31:21]

Patchset 1 (id:??) landed as
https://crrev.com/9972cd888b194adc1fc6de8bc6e53945846cb57e
Cr-Commit-Position: refs/heads/master@{#410437}