[Android] Attach RenderWidgetHostView to ContentView at construction time.

[Android] Attach RenderWidgetHostView to ContentView at construction time.

Upstreaming of downstream CL.

This handles the case where during ContentViewCore construction the
WebContents being used does not have a RWHV yet and it matches
other platforms where new windows in CreateViewForWidget() are shown
by default.

Original bugs:6937843,6931901

BUG=159063
TBR=ben

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=166620

[gone, 2012-11-07 02:19:12]

Just upstreaming 4 lines of diff.  Seems to help with rendering.

[no sievers, 2012-11-07 02:24:44]

lgtm

[commit-bot: I haz the power, 2012-11-07 18:36:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dfalcantara@chromium.org/11362130/1

[gone, 2012-11-07 18:37:20]

TBRing ben since it's an Android change.

[commit-bot: I haz the power, 2012-11-08 01:50:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dfalcantara@chromium.org/11362130/1

[commit-bot: I haz the power, 2012-11-08 06:31:08]

Change committed as 166620

[Philippe, 2012-11-08 18:10:06]

On 2012/11/08 06:31:08, I haz the power (commit-bot) wrote:
> Change committed as 166620

I think this makes ~80 instrumentation tests fail :)
See
http://build.chromium.org/p/chromium.fyi/builders/Android%20Tests%20%28JB%20N...

I did a bisect to track down this commit. Try r166619 (the commit before), it
doesn't fail.

[gone, 2012-11-08 22:22:28]

On 2012/11/08 18:10:06, Philippe wrote:
> On 2012/11/08 06:31:08, I haz the power (commit-bot) wrote:
> > Change committed as 166620
> 
> I think this makes ~80 instrumentation tests fail :)
> See
>
http://build.chromium.org/p/chromium.fyi/builders/Android%2520Tests%2520%2528...
> 
> I did a bisect to track down this commit. Try r166619 (the commit before), it
> doesn't fail.

Reverted it, but this works downstream and worked when it was first submitted. 
Don't know why it's broken now, but it seems like a bad interaction with
something that landed between this and Monday...

[gone, 2012-11-08 23:06:01]

On 2012/11/08 22:22:28, dfalcantara wrote:
> On 2012/11/08 18:10:06, Philippe wrote:
> > On 2012/11/08 06:31:08, I haz the power (commit-bot) wrote:
> > > Change committed as 166620
> > 
> > I think this makes ~80 instrumentation tests fail :)
> > See
> >
>
http://build.chromium.org/p/chromium.fyi/builders/Android%252520Tests%252520%...
> > 
> > I did a bisect to track down this commit. Try r166619 (the commit before),
it
> > doesn't fail.
> 
> Reverted it, but this works downstream and worked when it was first submitted.

> Don't know why it's broken now, but it seems like a bad interaction with
> something that landed between this and Monday...

Found the other culprit:
https://chromiumcodereview.appspot.com/11366118

When both of these land together, everything explodes.  Haven't figured out why,
yet.