Fixes and test updates for aura-mus

Fixes and test updates for aura-mus

This addresses the following:
. Wires up WindowPortMus::OnDeviceScaleFactorChanged() correctly.
. Properly routes DesktopWindowTreeHostMus::SetSize(). It needs to
  go to WindowTreeHost, which then updates window().
. Makes DesktopWindowTreeHostMus::Show/Hide call to the appropriate
  Widget functions
. Honors min/max size.

And updates some tests that are relying on specific NativeWidgetAura
functionality to create NativeWidgetAura.

BUG=660994
TEST=covered by tests
R=msw@chromium.org

Committed: https://crrev.com/2e7a168a82cebe5fbb355cb05401c4827758aa94
Cr-Commit-Position: refs/heads/master@{#430828}

[sky, 2016-11-08 22:07:14]

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

[commit-bot: I haz the power, 2016-11-08 22:08:01]

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

[msw, 2016-11-08 23:38:27]

lgtm with nits

https://codereview.chromium.org/2489513003/diff/1/ui/views/mus/desktop_window...
File ui/views/mus/desktop_window_tree_host_mus.cc (right):

https://codereview.chromium.org/2489513003/diff/1/ui/views/mus/desktop_window...
ui/views/mus/desktop_window_tree_host_mus.cc:128: // TODO: handle device scale.
optional nit: cite a bug? ditto on line 387

https://codereview.chromium.org/2489513003/diff/1/ui/views/mus/desktop_window...
ui/views/mus/desktop_window_tree_host_mus.cc:147: *bounds = GetRestoredBounds();
q: Why restored bounds? maybe comment?

https://codereview.chromium.org/2489513003/diff/1/ui/views/mus/desktop_window...
ui/views/mus/desktop_window_tree_host_mus.cc:375:
ShowWindowWithState(ui::SHOW_STATE_NORMAL);
q: Should this ever be showing with the prior state (eg. showing the window
after it was maximized, then hidden/minimized?)

https://codereview.chromium.org/2489513003/diff/1/ui/views/touchui/touch_sele...
File ui/views/touchui/touch_selection_menu_runner_views_unittest.cc (right):

https://codereview.chromium.org/2489513003/diff/1/ui/views/touchui/touch_sele...
ui/views/touchui/touch_selection_menu_runner_views_unittest.cc:35: // These
tests expect NativeWidgetAura, force it's creation.
nit: "its"

https://codereview.chromium.org/2489513003/diff/1/ui/views/touchui/touch_sele...
ui/views/touchui/touch_selection_menu_runner_views_unittest.cc:36: // TODO(sky):
the reason these end up using NativeWidgetAura is because they
optional nit: cite a bug?

https://codereview.chromium.org/2489513003/diff/1/ui/views/widget/widget_unit...
File ui/views/widget/widget_unittest.cc (right):

https://codereview.chromium.org/2489513003/diff/1/ui/views/widget/widget_unit...
ui/views/widget/widget_unittest.cc:850: // This test expect NativeWidgetAura,
force it's creation.
nit: "its"

[sky, 2016-11-09 00:06:01]

https://codereview.chromium.org/2489513003/diff/1/ui/views/mus/desktop_window...
File ui/views/mus/desktop_window_tree_host_mus.cc (right):

https://codereview.chromium.org/2489513003/diff/1/ui/views/mus/desktop_window...
ui/views/mus/desktop_window_tree_host_mus.cc:128: // TODO: handle device scale.
On 2016/11/08 23:38:27, msw wrote:
> optional nit: cite a bug? ditto on line 387

Done.

https://codereview.chromium.org/2489513003/diff/1/ui/views/mus/desktop_window...
ui/views/mus/desktop_window_tree_host_mus.cc:147: *bounds = GetRestoredBounds();
On 2016/11/08 23:38:27, msw wrote:
> q: Why restored bounds? maybe comment?

Done.

https://codereview.chromium.org/2489513003/diff/1/ui/views/mus/desktop_window...
ui/views/mus/desktop_window_tree_host_mus.cc:375:
ShowWindowWithState(ui::SHOW_STATE_NORMAL);
On 2016/11/08 23:38:27, msw wrote:
> q: Should this ever be showing with the prior state (eg. showing the window
> after it was maximized, then hidden/minimized?)

This matches that of DesktopWindowTreeHostX11. I added a comment.

https://codereview.chromium.org/2489513003/diff/1/ui/views/touchui/touch_sele...
File ui/views/touchui/touch_selection_menu_runner_views_unittest.cc (right):

https://codereview.chromium.org/2489513003/diff/1/ui/views/touchui/touch_sele...
ui/views/touchui/touch_selection_menu_runner_views_unittest.cc:35: // These
tests expect NativeWidgetAura, force it's creation.
On 2016/11/08 23:38:27, msw wrote:
> nit: "its"

Done.

https://codereview.chromium.org/2489513003/diff/1/ui/views/touchui/touch_sele...
ui/views/touchui/touch_selection_menu_runner_views_unittest.cc:36: // TODO(sky):
the reason these end up using NativeWidgetAura is because they
On 2016/11/08 23:38:27, msw wrote:
> optional nit: cite a bug?

Done.

https://codereview.chromium.org/2489513003/diff/1/ui/views/widget/widget_unit...
File ui/views/widget/widget_unittest.cc (right):

https://codereview.chromium.org/2489513003/diff/1/ui/views/widget/widget_unit...
ui/views/widget/widget_unittest.cc:850: // This test expect NativeWidgetAura,
force it's creation.
On 2016/11/08 23:38:27, msw wrote:
> nit: "its"

Done.

[sky, 2016-11-09 00:06:09]

The CQ bit was checked by sky@chromium.org

[sky, 2016-11-09 00:06:10]

The patchset sent to the CQ was uploaded after l-g-t-m from msw@chromium.org
Link to the patchset: https://codereview.chromium.org/2489513003/#ps20001
(title: "feedback")

[commit-bot: I haz the power, 2016-11-09 00:06:12]

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-11-09 02:09:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-09 02:15:17]

Description was changed from

==========
Fixes and test updates for aura-mus

This addresses the following:
. Wires up WindowPortMus::OnDeviceScaleFactorChanged() correctly.
. Properly routes DesktopWindowTreeHostMus::SetSize(). It needs to
  go to WindowTreeHost, which then updates window().
. Makes DesktopWindowTreeHostMus::Show/Hide call to the appropriate
  Widget functions
. Honors min/max size.

And updates some tests that are relying on specific NativeWidgetAura
functionality to create NativeWidgetAura.

BUG=660994
TEST=covered by tests
R=msw@chromium.org
==========

to

==========
Fixes and test updates for aura-mus

This addresses the following:
. Wires up WindowPortMus::OnDeviceScaleFactorChanged() correctly.
. Properly routes DesktopWindowTreeHostMus::SetSize(). It needs to
  go to WindowTreeHost, which then updates window().
. Makes DesktopWindowTreeHostMus::Show/Hide call to the appropriate
  Widget functions
. Honors min/max size.

And updates some tests that are relying on specific NativeWidgetAura
functionality to create NativeWidgetAura.

BUG=660994
TEST=covered by tests
R=msw@chromium.org

Committed: https://crrev.com/2e7a168a82cebe5fbb355cb05401c4827758aa94
Cr-Commit-Position: refs/heads/master@{#430828}
==========

[commit-bot: I haz the power, 2016-11-09 02:15:18]

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