Bug 36480

Summary:

Test that "load" doesn't appear on documents

Product:

WebKit

Reporter:

Adam Barth <abarth>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

NEW

Severity:

Normal

CC:

ap, gsnedders

Priority:

Version:

528+ (Nightly build)

Hardware:

Other

OS:

OS X 10.5

Bug Depends on:

Bug Blocks:

9063

Attachments:

Description	Flags
Patch	none
Patch	ap: review-

Adam Barth

Reported 2010-03-22 23:33:41 PDT

Test that "load" doesn't appear on HTMLDocument or SVGDocument

Attachments
Patch (2.64 KB, patch) 2010-03-22 23:35 PDT, Adam Barth	no flags	Details Formatted Diff Diff
Patch (3.67 KB, patch) 2010-03-23 00:41 PDT, Adam Barth	ap: review-	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Adam Barth

Comment 1 2010-03-22 23:35:13 PDT

Created attachment 51397 [details] Patch

Adam Barth

Comment 2 2010-03-23 00:41:22 PDT

Created attachment 51399 [details] Patch

Alexey Proskuryakov

Comment 3 2010-03-23 11:18:38 PDT

Comment on attachment 51399 [details] Patch As discussed in bug 9063, all documents have the same interface in HTML5. Maybe we should consider having document.load hidden, like document.all is.

Alexey Proskuryakov

Comment 4 2010-03-23 11:22:21 PDT

Whether this patch should be landed depends on whether we expect that Web will break if we visibly expose Document.load on some documents that don't have it in Firefox. Bug description doesn't mention such, so I r-'ed the patch for now.

Adam Barth

Comment 5 2010-03-23 13:17:54 PDT

This patch documents our existing behavior and links our behavior into the dependency tree of bugs where we're going to decide to keep or change that behavior. We can hold off landing the tests until we finish making up our minds, but it seems useful to document our current understanding.

Alexey Proskuryakov

Comment 6 2010-08-09 04:06:18 PDT

Renamed the bug to talk about non-XML documents. Per HTML5, all documents implement HTMLDocument and SVGDocument interfaces, but only XML documents implement XMLDocumentLoader.

Alexey Proskuryakov

Comment 7 2010-08-12 08:47:49 PDT

...and as of today, only documents created via createDocument() have it.

Sam Sneddon [:gsnedders]

Comment 8 2021-03-25 16:59:08 PDT

Note we're back to having separate interfaces, and load has been removed from every browser. Sure, we could still add a test for this (and a quick look doesn't find any in WPT), but it's exceptionally low priority.

Note You need to log in before you can comment on or make changes to this bug.