Initialize stub platform window bounds to those of its mus::Window.

Initialize stub platform window bounds to those of its mus::Window.

Fixes a regression from https://codereview.chromium.org/2009853002
IDK why it matters or if it relates to [still] broken window dragging.

TODO: Fix wallpaper picker dragging (not broken by same CL)...

BUG=557405
TEST=wallpaper picker shows for "Set wallpaper..." in mash.
R=markdittmer@chromium.org,sky@chromium.org,sadrul@chromium.org

Committed: https://crrev.com/6ac5be6880e84f4be61d7d8e63b1ff8283f7735c
Cr-Commit-Position: refs/heads/master@{#397873}

[msw, 2016-06-03 21:23:38]

Description was changed from

==========
Initialize stub platform window bounds to those of its mus::Window.

Fixes a regression from https://codereview.chromium.org/2009853002
IDK why it matters or if it relates to [still] broken window dragging.

BUG=557405
TEST=wallpaper picker shows for "Set wallpaper..." in mash.
R=markdittmer@chromium.org,sky@chromium.org,sadrul@chromium.org
==========

to

==========
Initialize stub platform window bounds to those of its mus::Window.

Fixes a regression from https://codereview.chromium.org/2009853002
IDK why it matters or if it relates to [still] broken window dragging.

TODO: Fix wallpaper picker dragging (not broken by same CL)...

BUG=557405
TEST=wallpaper picker shows for "Set wallpaper..." in mash.
R=markdittmer@chromium.org,sky@chromium.org,sadrul@chromium.org
==========

[msw, 2016-06-03 21:24:12]

Hey Mark, Scott, Sadrul, please take a look; thanks!
(wide distro for better ideas on how to fix this...)

[msw, 2016-06-03 21:24:20]

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

[commit-bot: I haz the power, 2016-06-03 21:25:07]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2037013003/1

[sky, 2016-06-03 22:12:52]

LGTM

https://codereview.chromium.org/2037013003/diff/1/ui/views/mus/window_tree_ho...
File ui/views/mus/window_tree_host_mus.cc (right):

https://codereview.chromium.org/2037013003/diff/1/ui/views/mus/window_tree_ho...
ui/views/mus/window_tree_host_mus.cc:58:
platform_window()->SetBounds(window->bounds());
This seems right to me. I see another bug, will file it separately.

[sky, 2016-06-03 22:39:21]

Actually, turned out not to be a bug but rather some unnecessary code.

On Fri, Jun 3, 2016 at 3:12 PM,  <sky@chromium.org> wrote:
> LGTM
>
>
>
https://codereview.chromium.org/2037013003/diff/1/ui/views/mus/window_tree_ho...
> File ui/views/mus/window_tree_host_mus.cc (right):
>
>
https://codereview.chromium.org/2037013003/diff/1/ui/views/mus/window_tree_ho...
> ui/views/mus/window_tree_host_mus.cc:58:
> platform_window()->SetBounds(window->bounds());
> This seems right to me. I see another bug, will file it separately.
>
> https://codereview.chromium.org/2037013003/

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2016-06-04 00:47:33]

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

[commit-bot: I haz the power, 2016-06-04 00:47:34]

Dry run: This issue passed the CQ dry run.

[msw, 2016-06-04 00:51:39]

The CQ bit was checked by msw@chromium.org

[commit-bot: I haz the power, 2016-06-04 00:52:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2037013003/1

[commit-bot: I haz the power, 2016-06-04 00:57:44]

Description was changed from

==========
Initialize stub platform window bounds to those of its mus::Window.

Fixes a regression from https://codereview.chromium.org/2009853002
IDK why it matters or if it relates to [still] broken window dragging.

TODO: Fix wallpaper picker dragging (not broken by same CL)...

BUG=557405
TEST=wallpaper picker shows for "Set wallpaper..." in mash.
R=markdittmer@chromium.org,sky@chromium.org,sadrul@chromium.org
==========

to

==========
Initialize stub platform window bounds to those of its mus::Window.

Fixes a regression from https://codereview.chromium.org/2009853002
IDK why it matters or if it relates to [still] broken window dragging.

TODO: Fix wallpaper picker dragging (not broken by same CL)...

BUG=557405
TEST=wallpaper picker shows for "Set wallpaper..." in mash.
R=markdittmer@chromium.org,sky@chromium.org,sadrul@chromium.org
==========

[commit-bot: I haz the power, 2016-06-04 00:57:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-04 01:00:41]

Description was changed from

==========
Initialize stub platform window bounds to those of its mus::Window.

Fixes a regression from https://codereview.chromium.org/2009853002
IDK why it matters or if it relates to [still] broken window dragging.

TODO: Fix wallpaper picker dragging (not broken by same CL)...

BUG=557405
TEST=wallpaper picker shows for "Set wallpaper..." in mash.
R=markdittmer@chromium.org,sky@chromium.org,sadrul@chromium.org
==========

to

==========
Initialize stub platform window bounds to those of its mus::Window.

Fixes a regression from https://codereview.chromium.org/2009853002
IDK why it matters or if it relates to [still] broken window dragging.

TODO: Fix wallpaper picker dragging (not broken by same CL)...

BUG=557405
TEST=wallpaper picker shows for "Set wallpaper..." in mash.
R=markdittmer@chromium.org,sky@chromium.org,sadrul@chromium.org

Committed: https://crrev.com/6ac5be6880e84f4be61d7d8e63b1ff8283f7735c
Cr-Commit-Position: refs/heads/master@{#397873}
==========

[commit-bot: I haz the power, 2016-06-04 01:00:42]

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