Bug 22372 - Add a test for HTMLIFrameElement.document
Summary: Add a test for HTMLIFrameElement.document
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Pam Greene (IRC:pamg)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-19 18:04 PST by Pam Greene (IRC:pamg)
Modified: 2008-11-20 09:52 PST (History)
1 user (show)

See Also:

Attachments
New test + result (1.65 KB, patch)
2008-11-19 18:10 PST, Pam Greene (IRC:pamg) 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Pam Greene (IRC:pamg) 2008-11-19 18:04:52 PST 
Test that if HTMLIFrameElement.document exists, it is an alias for HTMLIFrameElement.contentDocument.
Comment 1 Pam Greene (IRC:pamg) 2008-11-19 18:10:00 PST 
Created attachment 25297 [details]
New test + result
Comment 2 Sam Weinig 2008-11-19 20:56:43 PST 
Are you sure this correct?  HTMLIFrameElement.document and HTMLIFrameElement.contentDocument should not point to the same thing as far as I know.  HTMLIFrameElement.document should be the document the iframe is in, and HTMLIFrameElement.contentDocument should be the document in iframe.
Comment 3 Darin Adler 2008-11-20 07:45:13 PST 
Comment on attachment 25297 [details]
New test + result

r=me
Comment 4 Pam Greene (IRC:pamg) 2008-11-20 09:26:27 PST 
(In reply to comment #2)
> Are you sure this correct? 

No. Recent builds (3.2 and trunk) don't offer HTMLIFrameElement.document at all, and I can't find any additional documentation describing what it used to be.  I'll ask the original author of the test why he thought this was the intended behavior.

Removing Darin's r+ in the meantime.
Comment 5 Darin Adler 2008-11-20 09:36:14 PST 
LOL, sorry I blindly said review+ on this. I remember well when we removed this "document" attribute, and the terrible problems it was causing on many real world websites!
Comment 6 Pam Greene (IRC:pamg) 2008-11-20 09:52:10 PST 
Never mind, then. This test is no longer needed.