Put the logic to maximize/fullscreen window back in OnWindowAddedToLayout

Put the logic to maximize/fullscreen window back in OnWindowAddedToLayout

I'll look into if we can eliminate this.

BUG=343696
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251313

[hshi1, 2014-02-13 23:46:50]

lgtm

[oshima, 2014-02-13 23:47:29]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-02-13 23:51:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/165273002/100001

[pkotwicz, 2014-02-14 00:22:26]

https://codereview.chromium.org/165273002/diff/100001/ash/wm/workspace/worksp...
File ash/wm/workspace/workspace_layout_manager.cc (right):

https://codereview.chromium.org/165273002/diff/100001/ash/wm/workspace/worksp...
ash/wm/workspace/workspace_layout_manager.cc:129:
SetMaximizedOrFullscreenBounds(window_state);
Optional Nit: You could modify the implementation of
AdjustWindowBoundsWhenAdded() given that it is only invoked from here

[oshima, 2014-02-14 00:42:48]

https://codereview.chromium.org/165273002/diff/100001/ash/wm/workspace/worksp...
File ash/wm/workspace/workspace_layout_manager.cc (right):

https://codereview.chromium.org/165273002/diff/100001/ash/wm/workspace/worksp...
ash/wm/workspace/workspace_layout_manager.cc:129:
SetMaximizedOrFullscreenBounds(window_state);
On 2014/02/14 00:22:26, pkotwicz wrote:
> Optional Nit: You could modify the implementation of
> AdjustWindowBoundsWhenAdded() given that it is only invoked from here

Thanks, I'll clean up in separate CL with tests coverage.

[commit-bot: I haz the power, 2014-02-14 01:50:24]

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

[commit-bot: I haz the power, 2014-02-14 01:50:24]

List of reviewers changed. pkotwicz@chromium.org did a drive-by without
LGTM'ing!

[hshi1, 2014-02-14 01:50:58]

The CQ bit was checked by hshi@chromium.org

[commit-bot: I haz the power, 2014-02-14 01:52:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/165273002/100001

[commit-bot: I haz the power, 2014-02-14 04:09:51]

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

[commit-bot: I haz the power, 2014-02-14 04:09:52]

Try jobs failed on following builders: linux_chromium_rel

[oshima, 2014-02-14 04:17:00]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-02-14 04:19:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/165273002/100001

[commit-bot: I haz the power, 2014-02-14 05:17:29]

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

[commit-bot: I haz the power, 2014-02-14 05:17:30]

Try jobs failed on following builders: linux_chromium_rel

[Paweł Hajdan Jr., 2014-02-14 05:43:42]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2014-02-14 05:44:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/165273002/100001

[commit-bot: I haz the power, 2014-02-14 14:53:39]

Change committed as 251313