Disconnect frame owners when detaching frames.

Disconnect frame owners when detaching frames.

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

[dcheng, 2014-10-14 22:41:11]

dcheng@chromium.org changed reviewers:
+ japhet@chromium.org, sigbjorn@opera.com

[Nate Chapin, 2014-10-14 23:41:59]

LGTM

Is there anything to test here?

[dcheng, 2014-10-15 00:09:26]

On 2014/10/14 23:41:59, Nate Chapin wrote:
> LGTM
> 
> Is there anything to test here?

I was going to rebase this on top of your streamlining patch and eliminate the
logic duplicated into ChildFrameDisconnector/HTMLFrameOwnerElement.

Nasko hit this issue with his local patches, but we should also be able to
tickle this code with a WebFrameTest, so I'll give that a shot.

[haraken, 2014-10-15 00:50:51]

LGTM

[sof, 2014-10-15 12:23:54]

sigbjornf@opera.com changed reviewers:
+ sigbjornf@opera.com

[sof, 2014-10-15 12:23:55]

lgtm.

fwiw, we haven't come across Oilpan problems with having the owner disconnect
happen during detach.

[dcheng, 2014-10-16 00:38:01]

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

[dcheng, 2014-10-16 00:38:01]

PTAL. I've updated this patch on top of japhet@'s latest cleanup patch,
eliminated redundant codepaths, and added a test.

[dcheng, 2014-10-16 18:05:18]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2014-10-16 18:06:19]

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

[commit-bot: I haz the power, 2014-10-16 19:56:04]

Committed patchset #2 (id:20001) as 183834