ash: Remove unnecessary checks for null ShelfWidget in ShelfLayoutManager

ash: Remove unnecessary checks for null ShelfWidget in ShelfLayoutManager

The ShelfWidget exists for the lifetime of the ShelfLayoutManager. The
ShelfLayoutManager is created in the ShelfWidget's constructor, and exists
until the shelf container is destroyed at shutdown. Removing the checks makes
the lifetimes easier to understand.

BUG=615502
TEST=ash_unittests

Committed: https://crrev.com/37b9e6da90be32a6e163dbbdf6accb983203eff5
Cr-Commit-Position: refs/heads/master@{#413543}

[James Cook, 2016-08-22 20:35:43]

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

[commit-bot: I haz the power, 2016-08-22 20:36:38]

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

[James Cook, 2016-08-22 20:36:41]

jamescook@chromium.org changed reviewers:
+ msw@chromium.org

[James Cook, 2016-08-22 20:36:42]

msw, please take a look.

[msw, 2016-08-22 20:41:55]

lgtm

[commit-bot: I haz the power, 2016-08-22 21:36:02]

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

[commit-bot: I haz the power, 2016-08-22 21:36:03]

Dry run: This issue passed the CQ dry run.

[James Cook, 2016-08-22 21:37:00]

The CQ bit was checked by jamescook@chromium.org

[commit-bot: I haz the power, 2016-08-22 21:38:25]

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-22 21:46:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-22 21:48:18]

Description was changed from

==========
ash: Remove unnecessary checks for null ShelfWidget in ShelfLayoutManager

The ShelfWidget exists for the lifetime of the ShelfLayoutManager. The
ShelfLayoutManager is created in the ShelfWidget's constructor, and exists
until the shelf container is destroyed at shutdown. Removing the checks makes
the lifetimes easier to understand.

BUG=615502
TEST=ash_unittests
==========

to

==========
ash: Remove unnecessary checks for null ShelfWidget in ShelfLayoutManager

The ShelfWidget exists for the lifetime of the ShelfLayoutManager. The
ShelfLayoutManager is created in the ShelfWidget's constructor, and exists
until the shelf container is destroyed at shutdown. Removing the checks makes
the lifetimes easier to understand.

BUG=615502
TEST=ash_unittests

Committed: https://crrev.com/37b9e6da90be32a6e163dbbdf6accb983203eff5
Cr-Commit-Position: refs/heads/master@{#413543}
==========

[commit-bot: I haz the power, 2016-08-22 21:48:19]

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