Add check to identify the cause of the crash.

Add check to investigate the crash.

I run the clusterfuzz test locally but couldn't reproduce this crash. The crash can happen if the bubble gets closed after activation, so adding CHECK to investigate how that's happening.

BUG=651242
Committed: https://crrev.com/2bb40ba64accf3cdddb906a487f73b0f25afc49f
Cr-Commit-Position: refs/heads/master@{#423547}

[oshima, 2016-10-05 00:41:48]

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

[commit-bot: I haz the power, 2016-10-05 00:42:18]

Dry run: 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-10-05 01:21:57]

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

[commit-bot: I haz the power, 2016-10-05 01:21:58]

Dry run: This issue passed the CQ dry run.

[oshima, 2016-10-05 14:27:52]

Description was changed from

==========
Add check to identify the cause of the crash.

BUG=651242
==========

to

==========
Add check to investigate the crash.

BUG=651242
==========

[oshima, 2016-10-05 14:29:00]

Description was changed from

==========
Add check to investigate the crash.

BUG=651242
==========

to

==========
Add check to investigate the crash.

I run the clusterfuzz test locally but couldn't reproduce this crash. The crash
can happen if the bubble gets closed after activation, so adding CHECK to
investigate how that's happening.

BUG=651242
==========

[oshima, 2016-10-05 14:29:00]

oshima@chromium.org changed reviewers:
+ afakhry@chromium.org

[afakhry, 2016-10-05 23:32:02]

lgtm

https://codereview.chromium.org/2392183002/diff/1/ash/common/system/tray/syst...
File ash/common/system/tray/system_tray.h (right):

https://codereview.chromium.org/2392183002/diff/1/ash/common/system/tray/syst...
ash/common/system/tray/system_tray.h:286: // TODO(oshima): Remove this whwn
crbug.com/651242 is fixed.
Nit: whwn --> when

[oshima, 2016-10-06 15:20:59]

The CQ bit was checked by oshima@chromium.org

[oshima, 2016-10-06 15:21:00]

The patchset sent to the CQ was uploaded after l-g-t-m from afakhry@chromium.org
Link to the patchset: https://codereview.chromium.org/2392183002/#ps20001
(title: "Add check to identify the cause of the crash.")

[commit-bot: I haz the power, 2016-10-06 15:21:22]

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-10-06 15:49:50]

Description was changed from

==========
Add check to investigate the crash.

I run the clusterfuzz test locally but couldn't reproduce this crash. The crash
can happen if the bubble gets closed after activation, so adding CHECK to
investigate how that's happening.

BUG=651242
==========

to

==========
Add check to investigate the crash.

I run the clusterfuzz test locally but couldn't reproduce this crash. The crash
can happen if the bubble gets closed after activation, so adding CHECK to
investigate how that's happening.

BUG=651242
==========

[commit-bot: I haz the power, 2016-10-06 15:49:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-06 15:51:22]

Description was changed from

==========
Add check to investigate the crash.

I run the clusterfuzz test locally but couldn't reproduce this crash. The crash
can happen if the bubble gets closed after activation, so adding CHECK to
investigate how that's happening.

BUG=651242
==========

to

==========
Add check to investigate the crash.

I run the clusterfuzz test locally but couldn't reproduce this crash. The crash
can happen if the bubble gets closed after activation, so adding CHECK to
investigate how that's happening.

BUG=651242

Committed: https://crrev.com/2bb40ba64accf3cdddb906a487f73b0f25afc49f
Cr-Commit-Position: refs/heads/master@{#423547}
==========

[commit-bot: I haz the power, 2016-10-06 15:51:24]

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

[oshima, 2016-10-06 20:43:53]

https://codereview.chromium.org/2392183002/diff/1/ash/common/system/tray/syst...
File ash/common/system/tray/system_tray.h (right):

https://codereview.chromium.org/2392183002/diff/1/ash/common/system/tray/syst...
ash/common/system/tray/system_tray.h:286: // TODO(oshima): Remove this whwn
crbug.com/651242 is fixed.
On 2016/10/05 23:32:02, afakhry wrote:
> Nit: whwn --> when

Done.