Delay calling FrameOwner::setWidget when doing a remote->local frame transition

Delay calling FrameOwner::setWidget when doing a remote->local frame transition

If we call setWidget for a newly created LocalFrame that's not fully attached
to the frame tree, it will be in an inconsistent state and cause crashes during
painting. Instead, update the widget at swap-time, when the frame actually
becomes fully attached.

BUG=422583
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184313

[Nate Chapin, 2014-10-21 20:26:50]

japhet@chromium.org changed reviewers:
+ dcheng@chromium.org

[Nate Chapin, 2014-10-21 20:26:50]

Another quirk to get remote->local frame swap working :-/

[dcheng, 2014-10-23 20:42:14]

LGTM.

... this makes me so sad.

[Nate Chapin, 2014-10-23 20:42:47]

On 2014/10/23 20:42:14, dcheng wrote:
> LGTM.
> 
> ... this makes me so sad.

Yeah, sorry :(

[Nate Chapin, 2014-10-23 20:42:50]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2014-10-23 20:43:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/668963002/20001

[commit-bot: I haz the power, 2014-10-23 22:51:45]

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

[commit-bot: I haz the power, 2014-10-23 22:51:46]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/33358)

[Nate Chapin, 2014-10-23 22:54:55]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2014-10-23 22:57:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/668963002/20001

[commit-bot: I haz the power, 2014-10-23 23:40:50]

Committed patchset #2 (id:20001) as 184313