[Progress Events] Remove support for ProgressEvent.initProgressEvent() and Document.createEvent("ProgressEvent")

RESOLVED FIXED in mozilla22

Status

()

defect
RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: jungkee.song, Assigned: emk)

Tracking

({dev-doc-complete, site-compat})

unspecified
mozilla22
x86_64
Linux
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.17 (KHTML, like Gecko) Chrome/24.0.1312.70 Safari/537.17

Steps to reproduce:

ProgressEvent CR (http://www.w3.org/TR/progress-events/) test
- Test wiki: http://www.w3.org/wiki/Webapps/Interop/ProgressEvents
- Test suite: http://w3c-test.org/webapps/ProgressEvents/tests/
- Test case: http://w3c-test.org/webapps/ProgressEvents/tests/submissions/Ms2ger/constructor.html



Actual results:

Test case failed: Object created from ProgressEvent interface has a initProgressEvent() method.


Expected results:

Object created from ProgressEvent interface should not have initProgressEvent() method anymore.

initProgressEvent() had been present until
<http://www.w3.org/TR/2011/WD-progress-events-20110310/> version and removed
from <http://www.w3.org/TR/2011/WD-progress-events-20110809/> version. As
it's been replaced by constructor, it may not be supported any more.
Thanks for the cc, though I'm not sure why you cc'd me...

What do the WHATWG specs say on this?
(Assignee)

Comment 2

6 years ago
WHATWG spec no longer defines initProgressEvent, either.
http://xhr.spec.whatwg.org/#interface-progressevent
(Assignee)

Comment 3

6 years ago
Also, WebKit already removed initProgressEvent.
https://bugs.webkit.org/show_bug.cgi?id=71340
This bug is valid, but odd testcase. What if some "LegacyEvents" spec added initProgressEvent.
Implementing such spec without causing test failures wouldn't be possible.
If that happens, I'll happily update the test :)
(Assignee)

Comment 6

6 years ago
I left the function for C++ callers because the ProgressEvent constructor is hard to call from C++.
I didn't bump the iid because this will not change the binary layout.
Assignee: nobody → VYV03354
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #717343 - Flags: review?(bugs)
Comment on attachment 717343 [details] [diff] [review]
Make initProgressEvent() [noscript]

Do other browser have initProgressEvent?
Attachment #717343 - Flags: review?(bugs) → review+
(Assignee)

Comment 8

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/64d47c0e35a8
Ms2ger's test should be imported.

(In reply to Olli Pettay [:smaug] from comment #7)
> Do other browser have initProgressEvent?

Chrome does not support initProgressEvent (see comment #3).
IE10 has initProgressEvent, but it doesn't support ProgressEvent in the first place.
I don't care about dying Presto anymore.
Flags: in-testsuite?
(Assignee)

Comment 9

6 years ago
> it doesn't support ProgressEvent
it doesn't support ProgressEvent constructor
Comment on attachment 717343 [details] [diff] [review]
Make initProgressEvent() [noscript]

Actually, since we support .createEvent("progressevent"), we should have
initProgressEvent too.
So either remove both or keep them both.
Attachment #717343 - Flags: review+ → review-
(Assignee)

Updated

6 years ago
Whiteboard: [leave open]
Comment on attachment 717404 [details] [diff] [review]
Drop support for .createEvent("progressevent")

This all is tiny bit risky since IE supports this.
But better to try this now, early in cycle.
Attachment #717404 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/64d47c0e35a8
https://hg.mozilla.org/mozilla-central/rev/c2c1e378ab49
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Keywords: dev-doc-needed
Summary: [Progress Events] There must not be a initProgressEvent(). → [Progress Events] Remove support for ProgressEvent.initProgressEvent() and Document.createEvent("ProgressEvent")
I've added this bug to the compatibility doc. Please correct the info if wrong.
https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_22
Depends on: 870466
You need to log in before you can comment on or make changes to this bug.