exo: Do not call UpdateShelfStateForFullscreenChange() in ShellSurface

exo: Do not call UpdateShelfStateForFullscreenChange() in ShellSurface

ash::WorkspaceLayoutManager::OnPostWindowStateTypeChange() will call
UpdateShelfVisibility() to update shelf visibility for window state
changes for every window. So we don't need update shell visibility
for exo Window anymore.

BUG=670496
Review-Url: https://codereview.chromium.org/2687683004
Cr-Commit-Position: refs/heads/master@{#449110}
Committed: https://chromium.googlesource.com/chromium/src/+/ce3f7a721ca8dabb1b13ba506d9980d60c5e02ac

[Peng, 2017-02-08 18:16:29]

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

[commit-bot: I haz the power, 2017-02-08 18:16:52]

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

[Peng, 2017-02-08 18:19:11]

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

[Peng, 2017-02-08 18:19:12]

Hi Oshima, PTAL. Thanks.

[oshima, 2017-02-08 18:33:41]

non owner lgtm

I think you should also remove this as well.
https://cs.chromium.org/chromium/src/ash/common/wm/immersive_context_ash.cc?r...

[Peng, 2017-02-08 18:36:21]

On 2017/02/08 18:33:41, oshima wrote:
> non owner lgtm
> 
> I think you should also remove this as well.
>
https://cs.chromium.org/chromium/src/ash/common/wm/immersive_context_ash.cc?r...

I will investigate immersive_context_ash.cc later. Thanks.

[Peng, 2017-02-08 18:36:56]

penghuang@chromium.org changed reviewers:
+ reveman@chromium.org

[Peng, 2017-02-08 18:36:56]

Hi David, PTAL. Thanks.

[commit-bot: I haz the power, 2017-02-08 18:50:09]

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

[commit-bot: I haz the power, 2017-02-08 18:50:10]

Dry run: This issue passed the CQ dry run.

[reveman, 2017-02-08 21:58:28]

lgtm

[Peng, 2017-02-08 22:22:56]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2017-02-08 22:23:38]

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, 2017-02-08 22:29:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486592576556160, "parent_rev":
"925b3122a05d1ecaeedc8780a05089de19a6990a", "commit_rev":
"ce3f7a721ca8dabb1b13ba506d9980d60c5e02ac"}

[commit-bot: I haz the power, 2017-02-08 22:31:11]

Description was changed from

==========
exo: Do not call UpdateShelfStateForFullscreenChange() in ShellSurface

ash::WorkspaceLayoutManager::OnPostWindowStateTypeChange() will call
UpdateShelfVisibility() to update shelf visibility for window state
changes for every window. So we don't need update shell visibility
for exo Window anymore.

BUG=670496
==========

to

==========
exo: Do not call UpdateShelfStateForFullscreenChange() in ShellSurface

ash::WorkspaceLayoutManager::OnPostWindowStateTypeChange() will call
UpdateShelfVisibility() to update shelf visibility for window state
changes for every window. So we don't need update shell visibility
for exo Window anymore.

BUG=670496

Review-Url: https://codereview.chromium.org/2687683004
Cr-Commit-Position: refs/heads/master@{#449110}
Committed:
https://chromium.googlesource.com/chromium/src/+/ce3f7a721ca8dabb1b13ba506d99...
==========

[commit-bot: I haz the power, 2017-02-08 22:31:11]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ce3f7a721ca8dabb1b13ba506d99...