494705 - Deprecate DOM3 Load (XMLDocument.load)

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[timeless]

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....

Comment 1

[timeless]

Attachment: proposal (untested)

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

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.

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

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()

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

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.

Comment 5

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

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 :)

Comment 6

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Put it another way, grabbing 100 box-objects in rapid succession doesn't seem strange. Making 100 networks request seems much less likely.

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

Ok, perhaps the once-per-doc-warning isn't needed.

Comment 8

[timeless]

Attachment: patch

Comment 9

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Thanks timeless!

Comment 10

[Phil Ringnalda (:philor)]

http://hg.mozilla.org/mozilla-central/rev/9607c536d21a