Remove remote tree host and some related input and metro_driver code

Remove remote tree host and some related input and metro_driver code

Part of continued stripping out of Metro/Win8/Immersive mode.

BUG=558054
Committed: https://crrev.com/5cc04f70e424020d8afe49bdc4c1c76683f0ea7b
Cr-Commit-Position: refs/heads/master@{#370068}

[scottmg, 2016-01-13 23:19:16]

scottmg@chromium.org changed reviewers:
+ ananta@chromium.org, shuchen@chromium.org, sky@chromium.org,
thestig@chromium.org

[scottmg, 2016-01-13 23:19:17]

ash/, chrome/browser/, ui/aura/, ui/shell_dialogs/: sky@
printing/: thestig@
ui/base/ime/: shuchen@
win8/: ananta@

[scottmg, 2016-01-13 23:19:44]

Description was changed from

==========
Remove remote tree host and some related input and metro_driver code

BUG=558054
==========

to

==========
Remove remote tree host and some related input and metro_driver code

Part of continued stripping out of Metro/Win8/Immersive mode.

BUG=558054
==========

[Lei Zhang, 2016-01-13 23:35:32]

On 2016/01/13 23:19:17, scottmg wrote:
> printing/: thestig@

lgtm

[Shu Chen, 2016-01-14 02:41:58]

ui/base/ime lgtm.

[Shu Chen, 2016-01-14 07:52:06]

Just noticed that this cl will conflict with cl
https://codereview.chromium.org/1566083002, which will be merged to M48.
Can you please hold this cl until the cl
https://codereview.chromium.org/1566083002 is landed?
Thanks!

[sky, 2016-01-14 15:50:09]

https://codereview.chromium.org/1586843002/diff/1/chrome/browser/browser_proc...
File chrome/browser/browser_process_platform_part_aurawin.cc (right):

https://codereview.chromium.org/1586843002/diff/1/chrome/browser/browser_proc...
chrome/browser/browser_process_platform_part_aurawin.cc:13: void
BrowserProcessPlatformPart::PlatformSpecificCommandLineProcessing(
Seems like this class no longer has any content. Can it be nuked?

[scottmg, 2016-01-14 18:00:54]

On 2016/01/14 07:52:06, Shu Chen wrote:
> Just noticed that this cl will conflict with cl
> https://codereview.chromium.org/1566083002, which will be merged to M48.
> Can you please hold this cl until the cl
> https://codereview.chromium.org/1566083002 is landed?
> Thanks!

Yeah, no problem. Or is there a bit of this CL I could revert so that it
wouldn't conflict?

[scottmg, 2016-01-14 18:20:48]

https://codereview.chromium.org/1586843002/diff/1/chrome/browser/browser_proc...
File chrome/browser/browser_process_platform_part_aurawin.cc (right):

https://codereview.chromium.org/1586843002/diff/1/chrome/browser/browser_proc...
chrome/browser/browser_process_platform_part_aurawin.cc:13: void
BrowserProcessPlatformPart::PlatformSpecificCommandLineProcessing(
On 2016/01/14 15:50:09, sky wrote:
> Seems like this class no longer has any content. Can it be nuked?

Ah, I thought it needed a stub, but there's a common empty one used on Linux
too. Done.

[sky, 2016-01-14 20:39:31]

LGTM

[ananta, 2016-01-15 00:06:37]

metro stuff lgtm

[Shu Chen, 2016-01-15 01:22:54]

On 2016/01/14 18:00:54, scottmg wrote:
> On 2016/01/14 07:52:06, Shu Chen wrote:
> > Just noticed that this cl will conflict with cl
> > https://codereview.chromium.org/1566083002, which will be merged to M48.
> > Can you please hold this cl until the cl
> > https://codereview.chromium.org/1566083002 is landed?
> > Thanks!
> 
> Yeah, no problem. Or is there a bit of this CL I could revert so that it
> wouldn't conflict?

The deletion of ui/base/ime/remote_input_method_win.cc would cause the conflict.
Do you want to exclude that part from your cl?

[scottmg, 2016-01-15 03:40:46]

On 2016/01/15 01:22:54, Shu Chen wrote:
> On 2016/01/14 18:00:54, scottmg wrote:
> > On 2016/01/14 07:52:06, Shu Chen wrote:
> > > Just noticed that this cl will conflict with cl
> > > https://codereview.chromium.org/1566083002, which will be merged to M48.
> > > Can you please hold this cl until the cl
> > > https://codereview.chromium.org/1566083002 is landed?
> > > Thanks!
> > 
> > Yeah, no problem. Or is there a bit of this CL I could revert so that it
> > wouldn't conflict?
> 
> The deletion of ui/base/ime/remote_input_method_win.cc would cause the
conflict.
> Do you want to exclude that part from your cl?

That was the original thing I was trying to delete in this CL, so I guess I'll
just wait. Looks like your CL is close to landing anyway?

[scottmg, 2016-01-15 21:23:44]

scottmg@chromium.org changed reviewers:
+ dpranke@chromium.org

[scottmg, 2016-01-15 21:23:45]

+dpranke for testing/ (remove ash_unittests from some lists)

[Dirk Pranke, 2016-01-15 21:35:30]

lgtm

[scottmg, 2016-01-18 22:57:59]

Hi shuchen, do you still need me to hold off on this CL? I think the 48 stable
build train might have left the station.

[Shu Chen, 2016-01-19 00:22:50]

On 2016/01/18 22:57:59, scottmg wrote:
> Hi shuchen, do you still need me to hold off on this CL? I think the 48 stable
> build train might have left the station.

Thanks for holding this cl for me. Looks like my cl will miss M48 stable.
Please go ahead and land this cl.

[scottmg, 2016-01-19 00:28:13]

On 2016/01/19 00:22:50, Shu Chen wrote:
> On 2016/01/18 22:57:59, scottmg wrote:
> > Hi shuchen, do you still need me to hold off on this CL? I think the 48
stable
> > build train might have left the station.
> 
> Thanks for holding this cl for me. Looks like my cl will miss M48 stable.
> Please go ahead and land this cl.

OK, thanks.

[scottmg, 2016-01-19 00:28:19]

The CQ bit was checked by scottmg@chromium.org

[scottmg, 2016-01-19 00:28:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, shuchen@chromium.org, sky@chromium.org,
ananta@chromium.org
Link to the patchset: https://codereview.chromium.org/1586843002/#ps80001
(title: "remove ash_unittests from being run")

[commit-bot: I haz the power, 2016-01-19 00:28:28]

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

[commit-bot: I haz the power, 2016-01-19 01:40:38]

Description was changed from

==========
Remove remote tree host and some related input and metro_driver code

Part of continued stripping out of Metro/Win8/Immersive mode.

BUG=558054
==========

to

==========
Remove remote tree host and some related input and metro_driver code

Part of continued stripping out of Metro/Win8/Immersive mode.

BUG=558054
==========

[commit-bot: I haz the power, 2016-01-19 01:40:39]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-01-19 01:41:56]

Description was changed from

==========
Remove remote tree host and some related input and metro_driver code

Part of continued stripping out of Metro/Win8/Immersive mode.

BUG=558054
==========

to

==========
Remove remote tree host and some related input and metro_driver code

Part of continued stripping out of Metro/Win8/Immersive mode.

BUG=558054

Committed: https://crrev.com/5cc04f70e424020d8afe49bdc4c1c76683f0ea7b
Cr-Commit-Position: refs/heads/master@{#370068}
==========

[commit-bot: I haz the power, 2016-01-19 01:41:57]

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

[samli, 2016-01-20 03:05:29]

A revert of this CL (patchset #5 id:80001) has been created in
https://codereview.chromium.org/1602403002/ by samli@chromium.org.

The reason for reverting is: Suspecting that this is breaking the Webkit Win
(dbg) builder.

First failed build:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win%20Builder%....