Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

This is needed for mash as we asynchronously set the shelf, which is not expected. The async creation is temporary while we support sysui. Once sysui goes away this code shouldn't be needed.

BUG=632099
TEST=none
R=jamescook@chromium.org

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

[sky, 2016-07-27 19:17:56]

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

[sky, 2016-07-27 19:18:03]

Description was changed from

==========
Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

BUG=none
TEST=none
R=jamescook@chromium.org
==========

to

==========
Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

BUG=632099
TEST=none
R=jamescook@chromium.org
==========

[commit-bot: I haz the power, 2016-07-27 19:18:19]

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

[sky, 2016-07-27 19:18:42]

Description was changed from

==========
Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

BUG=632099
TEST=none
R=jamescook@chromium.org
==========

to

==========
Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

This is needed for mash as we asynchronously set the shelf, which is not
expected. The async creation is temporary while we support sysui. Once sysui
goes away this code shouldn't be needed.

BUG=632099
TEST=none
R=jamescook@chromium.org
==========

[sky, 2016-07-27 19:19:15]

I updated the description to reflect this is temporary and added a pointer to
the bug.

[James Cook, 2016-07-27 19:43:53]

LGTM

[sky, 2016-07-27 19:45:13]

The CQ bit was unchecked by sky@chromium.org

[sky, 2016-07-27 19:45:17]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-07-27 19:45: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-07-27 19:55:31]

Description was changed from

==========
Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

This is needed for mash as we asynchronously set the shelf, which is not
expected. The async creation is temporary while we support sysui. Once sysui
goes away this code shouldn't be needed.

BUG=632099
TEST=none
R=jamescook@chromium.org
==========

to

==========
Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

This is needed for mash as we asynchronously set the shelf, which is not
expected. The async creation is temporary while we support sysui. Once sysui
goes away this code shouldn't be needed.

BUG=632099
TEST=none
R=jamescook@chromium.org
==========

[commit-bot: I haz the power, 2016-07-27 19:55:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-27 19:58:35]

Description was changed from

==========
Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

This is needed for mash as we asynchronously set the shelf, which is not
expected. The async creation is temporary while we support sysui. Once sysui
goes away this code shouldn't be needed.

BUG=632099
TEST=none
R=jamescook@chromium.org
==========

to

==========
Makes DockedWindowLayoutManager::UpdateDockBounds() handle null shelf_

This is needed for mash as we asynchronously set the shelf, which is not
expected. The async creation is temporary while we support sysui. Once sysui
goes away this code shouldn't be needed.

BUG=632099
TEST=none
R=jamescook@chromium.org

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

[commit-bot: I haz the power, 2016-07-27 19:58:36]

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