Deprecate DOM3 Load (XMLDocument.load)

RESOLVED FIXED in mozilla1.9.3a5

Status

()

defect
RESOLVED FIXED
10 years ago
3 months ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

Trunk
mozilla1.9.3a5
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

2.16 KB, patch
timeless
: review+
Details | Diff | Splinter Review
Assignee

Description

10 years ago
https://bugs.webkit.org/show_bug.cgi?id=9063 indicates web sites are suffering because of an old dom3 api proposal that we implemented.

Given that the web has chosen to use XMLHttpRequest instead, I'd like to start the deprecation process for this function....
Assignee

Comment 1

10 years ago
Posted patch proposal (untested) (obsolete) — Splinter Review
Attachment #379476 - Flags: review?(jst)
Not sure what to do with this.
https://bugs.webkit.org/show_bug.cgi?id=9063#c9

Though, I think I'd prefer deprecating this.
Note that this was something we inherited from IE, not an effort to implement DOM3 L&S.

However I'm all for trying to deprecate this, though I'm not convinced we'll be successful.

So r+=me on the patch if you change the error to mention Document.load() rather than just load()
Also, we may want to give the warning message only once per doc or something.
Otherwise error console may be full of warnings.
We do something similar with boxobjects.
I suspect getting box-objects happened a lot more than loading things off the network though, so probably not as important here. In fact, I almost like people getting spammed if it makes people notice that the API is deprecated :)
Put it another way, grabbing 100 box-objects in rapid succession doesn't seem strange. Making 100 networks request seems much less likely.
Ok, perhaps the once-per-doc-warning isn't needed.
Assignee

Comment 8

9 years ago
Posted patch patchSplinter Review
Attachment #379476 - Attachment is obsolete: true
Attachment #438644 - Flags: review+
Attachment #379476 - Flags: review?(jst)
Assignee

Updated

9 years ago
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/9607c536d21a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite-
Keywords: checkin-needed
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Component: DOM: Mozilla Extensions → DOM

Updated

11 months ago
Blocks: 332175
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.