Issue FrameDestructionObserver::frameDestroyed() notification on detach.

LayoutTests/fast/dom/Window/property-access-on-cached-window-after-frame-removed-expected.txt

 CONSOLE WARNING: 'window.webkitStorageInfo' is deprecated. Please use 'navigator.webkitTemporaryStorage' or 'navigator.webkitPersistentStorage' instead.
 CONSOLE WARNING: 'webkitIndexedDB' is deprecated. Please use 'indexedDB' instead.
 Tests property access on a cached DOMWindow after the associated frame is no longer in a web page. Test should not crash and properties should be set to sane defaults.
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
 FAIL childWindow.applicationCache.oncached should be null. Threw exception TypeError: Cannot read property 'oncached' of null
 FAIL childWindow.applicationCache.onchecking should be null. Threw exception TypeError: Cannot read property 'onchecking' of null
 FAIL childWindow.applicationCache.ondownloading should be null. Threw exception TypeError: Cannot read property 'ondownloading' of null
 FAIL childWindow.applicationCache.onerror should be null. Threw exception TypeError: Cannot read property 'onerror' of null
 FAIL childWindow.applicationCache.onnoupdate should be null. Threw exception TypeError: Cannot read property 'onnoupdate' of null
 FAIL childWindow.applicationCache.onobsolete should be null. Threw exception TypeError: Cannot read property 'onobsolete' of null
 FAIL childWindow.applicationCache.onprogress should be null. Threw exception TypeError: Cannot read property 'onprogress' of null
 FAIL childWindow.applicationCache.onupdateready should be null. Threw exception TypeError: Cannot read property 'onupdateready' of null
 FAIL childWindow.applicationCache.status should be 0. Threw exception TypeError: Cannot read property 'status' of null
 PASS childWindow.closed is true
 PASS childWindow.defaultStatus is ''
 PASS childWindow.defaultstatus is ''
 PASS childWindow.devicePixelRatio is 0
 PASS childWindow.frameElement is null
 PASS childWindow.history.length is 0
 PASS childWindow.history.state is null
 PASS childWindow.innerHeight is 0
 PASS childWindow.innerWidth is 0
 FAIL childWindow.isJsTest should be false (of type boolean). Was undefined (of type undefined).
 PASS childWindow.length is 0
-PASS childWindow.location.ancestorOrigins.length is 0
-PASS childWindow.location.hash is ''
-PASS childWindow.location.host is ''
-PASS childWindow.location.hostname is ''
-PASS childWindow.location.href is 'about:blank'
-PASS childWindow.location.origin is 'null'
-PASS childWindow.location.pathname is 'blank'
-PASS childWindow.location.port is ''
-PASS childWindow.location.protocol is 'about:'
-PASS childWindow.location.search is ''
+FAIL childWindow.location.ancestorOrigins.length should be 0. Threw exception TypeError: Cannot read property 'length' of undefined
+FAIL childWindow.location.hash should be  (of type string). Was undefined (of type undefined).
+FAIL childWindow.location.host should be  (of type string). Was undefined (of type undefined).
+FAIL childWindow.location.hostname should be  (of type string). Was undefined (of type undefined).
+FAIL childWindow.location.href should be about:blank (of type string). Was undefined (of type undefined).
+FAIL childWindow.location.origin should be null (of type string). Was undefined (of type undefined).
+FAIL childWindow.location.pathname should be blank (of type string). Was undefined (of type undefined).
+FAIL childWindow.location.port should be  (of type string). Was undefined (of type undefined).
+FAIL childWindow.location.protocol should be about: (of type string). Was undefined (of type undefined).
+FAIL childWindow.location.search should be  (of type string). Was undefined (of type undefined).
 PASS childWindow.locationbar.visible is false
 PASS childWindow.menubar.visible is false
 PASS childWindow.name is ''
 PASS childWindow.navigator.appCodeName is window.navigator.appCodeName
 PASS childWindow.navigator.appName is window.navigator.appName
 PASS childWindow.navigator.appVersion is ''
 PASS childWindow.navigator.connection.ontypechange is null
 PASS childWindow.navigator.cookieEnabled is false
 PASS childWindow.navigator.doNotTrack is null
 PASS childWindow.navigator.hardwareConcurrency is window.navigator.hardwareConcurrency
@@ skipping 127 matching lines @@
 PASS childWindow.performance.timing.secureConnectionStart is 0
 PASS childWindow.performance.timing.unloadEventEnd is 0
 PASS childWindow.performance.timing.unloadEventStart is 0
 PASS childWindow.personalbar.visible is false
 PASS childWindow.screen.availHeight is 0
 PASS childWindow.screen.availLeft is 0
 PASS childWindow.screen.availTop is 0
 PASS childWindow.screen.availWidth is 0
 PASS childWindow.screen.colorDepth is 0
 PASS childWindow.screen.height is 0
-PASS childWindow.screen.orientation.angle is 0
-PASS childWindow.screen.orientation.onchange is null
-FAIL childWindow.screen.orientation.type should be . Was portrait-primary.
+FAIL childWindow.screen.orientation.angle should be 0. Threw exception TypeError: Cannot read property 'angle' of null
+FAIL childWindow.screen.orientation.onchange should be null. Threw exception TypeError: Cannot read property 'onchange' of null
+FAIL childWindow.screen.orientation.type should be . Threw exception TypeError: Cannot read property 'type' of null
 PASS childWindow.screen.pixelDepth is 0
 PASS childWindow.screen.width is 0
 PASS childWindow.screenLeft is 0
 PASS childWindow.screenTop is 0
 PASS childWindow.screenX is 0
 PASS childWindow.screenY is 0
 PASS childWindow.scrollX is 0
 PASS childWindow.scrollY is 0
 PASS childWindow.scrollbars.visible is false
-PASS childWindow.speechSynthesis.onvoiceschanged is null
-PASS childWindow.speechSynthesis.paused is false
-PASS childWindow.speechSynthesis.pending is false
-PASS childWindow.speechSynthesis.speaking is false
+FAIL childWindow.speechSynthesis.onvoiceschanged should be null. Threw exception TypeError: Cannot read property 'onvoiceschanged' of null
+FAIL childWindow.speechSynthesis.paused should be false. Threw exception TypeError: Cannot read property 'paused' of null
+FAIL childWindow.speechSynthesis.pending should be false. Threw exception TypeError: Cannot read property 'pending' of null
+FAIL childWindow.speechSynthesis.speaking should be false. Threw exception TypeError: Cannot read property 'speaking' of null

[haraken, 2015/01/31 01:30:01]

Does the new behavior align with the spec and other browsers?

If so, we might want to update the test so that the result becomes PASS.

[dcheng, 2015/01/31 01:38:14]

I don't think this area is specced out well. I have a small test page, and in
Firefox, window.location on a detached window returns null. So there's some
precedent for making these inaccessible on frame detach.

However, the behavior across different properties is not consistent. For
example, window.name seems to persist on a detached window.

I have a long-term goal to make Blink clean up the DOMWindow::m_frame back
pointer more aggressively (on DOMWindow detach, rather than on DOMWindow
destruction). This implies more things will be null/undefined after the window
is detached, but I haven't gotten around to sending an email to whatwg@ to see
if this is feasible or desirable.

[sof, 2015/01/31 08:07:23]

The lack of "childWindow.location" here is due to the binding security check for
Frames; it returning false when there being no frame.

All else being equal, it would be preferable to keep ".location" accessible, but
I'm not sure we want to tweak that security check.

fwiw, the location behavior now aligns with Gecko (but not Presto nor Trident),
but this is less-than-well-defined terrain spec-wise.

[sof, 2015/01/31 08:13:55]

Also, the use of "FAIL" for these property-collector tests is a bit non-standard
(at least that's how I think of the FAIL usage here) -- they're in the sense of
"not the same value as before a frame/window operation" rather than
"globally&inherently wrong".

cf. expected applicationCache results for this particular variant.

 PASS childWindow.status is ''
 PASS childWindow.statusbar.visible is false
 PASS childWindow.styleMedia.type is ''
 PASS childWindow.toolbar.visible is false
 PASS successfullyParsed is true
 
 TEST COMPLETE