Add WindowTreeHostPlatform::GetHost for Android build

Add WindowTreeHostPlatform::GetHost for Android build

BUG=507792

[bshe, 2015-10-23 20:27:36]

bshe@chromium.org changed reviewers:
+ mfomitchev@chromium.org

[bshe, 2015-10-23 20:27:36]

Hi Mikhail.

Following your suggestion, I uploaded this CL. I am not sure if we will end up
just create one WTH for aura Android. But upload it anyway so we could have
discussion here. Note that https://codereview.chromium.org/1421083003/ will be
depend on this CL.

Thanks!

[mfomitchev, 2015-10-23 21:17:00]

LGTM.
I think this is fine for now. We will need something better once we need to
support multiple windows, but that work can happen in TOT.

[mfomitchev, 2015-10-29 15:53:08]

mfomitchev@chromium.org changed reviewers:
+ sadrul@chromium.org, sievers@chromium.org

[sadrul, 2015-10-29 15:57:56]

We shouldn't add this. Where do you need it?

[mfomitchev, 2015-10-29 16:02:37]

On 2015/10/29 15:57:56, sadrul wrote:
> We shouldn't add this. Where do you need it?

See this: https://codereview.chromium.org/1321913004/
Also useful here: https://codereview.chromium.org/1421083003/

[mfomitchev, 2015-10-29 16:03:23]

Description was changed from

==========
Add WindowTreeHostPlatform::GetHost for Android build

BUG=507792
==========

to

==========
Add WindowTreeHostPlatform::GetHost for Android build

BUG=507792
==========

[no sievers, 2015-10-29 18:02:08]

but there is one WTH for each popup window (incl. menu) right? so it wouldn't
work

[mfomitchev, 2015-10-29 18:15:54]

On 2015/10/29 18:02:08, sievers wrote:
> but there is one WTH for each popup window (incl. menu) right? so it wouldn't
> work

I *think* we only create a new host in DesktopNativeWidgetAura, and we are not
using DesktopNativeWidgetAura currently. I made a change in ChromeViewsDelegate
to allocate NativeWidgetAura instead of DesktopNativeWidgetAura.

[no sievers, 2015-10-29 18:19:58]

On 2015/10/29 18:15:54, mfomitchev wrote:
> On 2015/10/29 18:02:08, sievers wrote:
> > but there is one WTH for each popup window (incl. menu) right? so it
wouldn't
> > work
> 
> I *think* we only create a new host in DesktopNativeWidgetAura, and we are not
> using DesktopNativeWidgetAura currently. I made a change in
ChromeViewsDelegate
> to allocate NativeWidgetAura instead of DesktopNativeWidgetAura.

But that would still be broken then, since we need a new host (or rather
compositor) for all free-floating windows.

[mfomitchev, 2015-10-29 18:39:41]

On 2015/10/29 18:19:58, sievers wrote:
> On 2015/10/29 18:15:54, mfomitchev wrote:
> > On 2015/10/29 18:02:08, sievers wrote:
> > > but there is one WTH for each popup window (incl. menu) right? so it
> wouldn't
> > > work
> > 
> > I *think* we only create a new host in DesktopNativeWidgetAura, and we are
not
> > using DesktopNativeWidgetAura currently. I made a change in
> ChromeViewsDelegate
> > to allocate NativeWidgetAura instead of DesktopNativeWidgetAura.
> 
> But that would still be broken then, since we need a new host (or rather
> compositor) for all free-floating windows.

Yeah, it won't work for multi-window. I thought it would be okay for now, and
then we could fix this up when we add multiwindow support..

[bshe, 2015-10-30 14:49:50]

On 2015/10/29 18:39:41, mfomitchev wrote:
> On 2015/10/29 18:19:58, sievers wrote:
> > On 2015/10/29 18:15:54, mfomitchev wrote:
> > > On 2015/10/29 18:02:08, sievers wrote:
> > > > but there is one WTH for each popup window (incl. menu) right? so it
> > wouldn't
> > > > work
> > > 
> > > I *think* we only create a new host in DesktopNativeWidgetAura, and we are
> not
> > > using DesktopNativeWidgetAura currently. I made a change in
> > ChromeViewsDelegate
> > > to allocate NativeWidgetAura instead of DesktopNativeWidgetAura.
> > 
> > But that would still be broken then, since we need a new host (or rather
> > compositor) for all free-floating windows.
> 
> Yeah, it won't work for multi-window. I thought it would be okay for now, and
> then we could fix this up when we add multiwindow support..

Close this issue without submit since it is no longer needed by
https://codereview.chromium.org/1421083003/.
We dont want to add a globe WindowTreeHostPlatform::GetHost.