Move Show and Hide impls from PlatformWindowMus to NativeWidgetMus

Move Show and Hide impls from PlatformWindowMus to NativeWidgetMus

R=sadrul@chromium.org
BUG=609555
Committed: https://crrev.com/1462909717a364b71c6291e742b2fc763b69b849
Cr-Commit-Position: refs/heads/master@{#392615}

[Mark Dittmer, 2016-05-09 15:49:45]

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

[commit-bot: I haz the power, 2016-05-09 15:50:05]

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

[Mark Dittmer, 2016-05-09 16:01:37]

DO NOT COMMIT: Test code checks for unexpected Show/Hide calls

[Mark Dittmer, 2016-05-09 16:01:45]

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

[commit-bot: I haz the power, 2016-05-09 16:02:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1962043002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1962043002/20001

[Mark Dittmer, 2016-05-09 16:02:10]

On 2016/05/09 16:01:37, Mark Dittmer wrote:
> DO NOT COMMIT: Test code checks for unexpected Show/Hide calls

sadrul@ PTAL at Patch Set 1.

[sadrul, 2016-05-09 16:38:26]

https://codereview.chromium.org/1962043002/diff/1/ui/views/mus/platform_windo...
File ui/views/mus/platform_window_mus.cc (right):

https://codereview.chromium.org/1962043002/diff/1/ui/views/mus/platform_windo...
ui/views/mus/platform_window_mus.cc:132: NOTIMPLEMENTED();
Add a class-level comment that PlatformWindowMus is going to be removed, and
remove the NOTIMPLEMENTED()s from here.

[commit-bot: I haz the power, 2016-05-09 17:23:22]

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

[commit-bot: I haz the power, 2016-05-09 17:23:22]

Dry run: Try jobs failed on following builders:
  win_clang on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[Mark Dittmer, 2016-05-09 17:53:10]

Drop WIP testing code

[Mark Dittmer, 2016-05-09 17:53:42]

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

[commit-bot: I haz the power, 2016-05-09 17:54:16]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1962043002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1962043002/40001

[commit-bot: I haz the power, 2016-05-09 18:34:17]

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

[commit-bot: I haz the power, 2016-05-09 18:34:18]

Dry run: This issue passed the CQ dry run.

[Mark Dittmer, 2016-05-09 19:03:57]

rebase

[Mark Dittmer, 2016-05-09 19:04:44]

All comments addressed.

https://codereview.chromium.org/1962043002/diff/1/ui/views/mus/platform_windo...
File ui/views/mus/platform_window_mus.cc (right):

https://codereview.chromium.org/1962043002/diff/1/ui/views/mus/platform_windo...
ui/views/mus/platform_window_mus.cc:132: NOTIMPLEMENTED();
On 2016/05/09 16:38:26, sadrul wrote:
> Add a class-level comment that PlatformWindowMus is going to be removed, and
> remove the NOTIMPLEMENTED()s from here.

Addressed in https://codereview.chromium.org/1953293004/ and here.

[sadrul, 2016-05-09 20:20:48]

lgtm

[Mark Dittmer, 2016-05-10 13:31:23]

Drop use of NOTIMPLEMENTED

[Mark Dittmer, 2016-05-10 14:31:34]

rebase

[Mark Dittmer, 2016-05-10 14:32:01]

The CQ bit was checked by markdittmer@chromium.org

[Mark Dittmer, 2016-05-10 14:32:02]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/1962043002/#ps100001
(title: "rebase")

[commit-bot: I haz the power, 2016-05-10 14:32:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1962043002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1962043002/100001

[commit-bot: I haz the power, 2016-05-10 15:32:01]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-05-10 15:33:08]

Description was changed from

==========
Move Show and Hide impls from PlatformWindowMus to NativeWidgetMus

R=sadrul@chromium.org
BUG=609555
==========

to

==========
Move Show and Hide impls from PlatformWindowMus to NativeWidgetMus

R=sadrul@chromium.org
BUG=609555

Committed: https://crrev.com/1462909717a364b71c6291e742b2fc763b69b849
Cr-Commit-Position: refs/heads/master@{#392615}
==========

[commit-bot: I haz the power, 2016-05-10 15:33:10]

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