Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upload.onProgress in IE10 #10038

Closed
DartBot opened this issue Apr 19, 2013 · 6 comments
Closed

upload.onProgress in IE10 #10038

DartBot opened this issue Apr 19, 2013 · 6 comments
Assignees
Labels
web-libraries Issues impacting dart:html, etc., libraries
Milestone

Comments

@DartBot
Copy link

DartBot commented Apr 19, 2013

This issue was originally filed by DrCol...@gmail.com


What steps will reproduce the problem?

Using upload.onProgress

e.g.
HttpRequest request = new HttpRequest();
request.upload.onProgress.listen((e) {
     
});

I know that old versions (inlcuding IE9) do not support this. But IE10 is supposted to support this event. See on SO http://stackoverflow.com/a/12866419/368003

What is the expected output? What do you see instead?
Proper processing of the event. The exception simply says

"onprogress is not a supported method on the object".

When i use the wonderfull (...) debugging dev tools, I can see that the XMLHTTPRequest Object has onprogress on it, so the transition from Dart to IE does seem to address the false method/property/call/...?

What version of the product are you using? On what operating system?
IE10, Win7, latest Dart build

@iposva-google
Copy link
Contributor

Added Area-HTML, Triaged labels.

@blois
Copy link
Contributor

blois commented Apr 22, 2013

The event is supported on HttpRequest, I'm not sure that it is fully supported in IE on the HttpRequestUpload object.

This syntax is tested & supported:
HttpRequest request = new HttpRequest();
request.onProgress.listen((e) {
});


Added this to the M5 milestone.

@DartBot
Copy link
Author

DartBot commented Apr 23, 2013

This comment was originally written by DrColosso...@gmail.com


No, this works even less. Now Chrome and co don't work as well. When using request.upload.onProress, I got results on some browsers, now I'm not able to see anything.

@blois
Copy link
Contributor

blois commented Jun 5, 2013

Set owner to @blois.
Added Accepted label.

@blois
Copy link
Contributor

blois commented Jun 5, 2013

I'm seeing this work properly on Chrome & FF, and have a fix in progress for IE.

One thing though is that the progress event only seems to fire when the data is sufficiently large that it doesn't all get sent at once.

@blois
Copy link
Contributor

blois commented Jun 5, 2013

Fixed with https://codereview.chromium.org/16376006/


Added Fixed label.

@DartBot DartBot added Type-Defect web-libraries Issues impacting dart:html, etc., libraries labels Jun 5, 2013
@DartBot DartBot added this to the M5 milestone Jun 5, 2013
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
web-libraries Issues impacting dart:html, etc., libraries
Projects
None yet
Development

No branches or pull requests

3 participants