Commit da7cd15c authored by darin's avatar darin
Browse files

Reviewed by Tim Hatcher.

        - fix <rdar://problem/5206089> REGRESSION: hovering over amazon.com "product categories"
          tab makes page content disappear

        The contentWindow property was doing a cross-frame security check, but it shouldn't.
        Window objects are responsible for protecting themselves from cross-frame activity.
        Because contentWindow returned undefined, we then ended up setting visibility on the
        body of the main frame to "hidden".

        No layout test yet. When we make cross-domain tests, we should include this one.

        * html/HTMLFrameElement.idl: Removed CheckFrameSecurity attribute from contentWindow.
        * html/HTMLIFrameElement.idl: Ditto.



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@21646 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent e3681b65
2007-05-22 Darin Adler <darin@apple.com>
Reviewed by Tim Hatcher.
- fix <rdar://problem/5206089> REGRESSION: hovering over amazon.com "product categories"
tab makes page content disappear
The contentWindow property was doing a cross-frame security check, but it shouldn't.
Window objects are responsible for protecting themselves from cross-frame activity.
Because contentWindow returned undefined, we then ended up setting visibility on the
body of the main frame to "hidden".
No layout test yet. When we make cross-domain tests, we should include this one.
* html/HTMLFrameElement.idl: Removed CheckFrameSecurity attribute from contentWindow.
* html/HTMLIFrameElement.idl: Ditto.
2007-05-22 Justin Garcia <justin.garcia@apple.com>
 
Reviewed by oliver
......@@ -37,7 +37,7 @@ module html {
// Extensions
#if defined(LANGUAGE_JAVASCRIPT)
// FIXME: this need not be JS only. See http://bugs.webkit.org/show_bug.cgi?id=13798
readonly attribute [CheckFrameSecurity] Window contentWindow;
readonly attribute Window contentWindow;
#endif
attribute [ConvertNullToNullString] DOMString location;
......
......@@ -39,7 +39,7 @@ module html {
// Extensions
#if defined(LANGUAGE_JAVASCRIPT)
// FIXME: this need not be JS only. See http://bugs.webkit.org/show_bug.cgi?id=13798
readonly attribute [CheckFrameSecurity] Window contentWindow;
readonly attribute Window contentWindow;
#endif
};
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment