Skip to content

Error with _AttributeMap in Firefox #11879

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

Closed
danschultz opened this issue Jul 17, 2013 · 5 comments
Closed

Error with _AttributeMap in Firefox #11879

danschultz opened this issue Jul 17, 2013 · 5 comments
Labels
closed-obsolete Closed as the reported issue is no longer relevant web-libraries Issues impacting dart:html, etc., libraries

Comments

@danschultz
Copy link

Iterating over the keys of an element's attributes is throwing an error in Firefox. Note: https://codereview.chromium.org/18302005/ does not fix this issue.

What is the expected output? What do you see instead?
To not have an error. Instead the following happens:

uncaught exception: NoSuchMethodError : method not found: 'length'
Receiver: Instance of 'JSUnknown'
Arguments: []
.wrapException@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:31163
.Interceptor.noSuchMethod$1@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:26585
.Object.get$length@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:59790
._AttributeMap.get$keys@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:72279
._AttributeMap.forEach$1@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:72266
$.forEach$1$ax@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:124158
._DataAttributeMap.forEach$1@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:72452
$.forEach$1$ax@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:124158
.HashMap.addAll$1@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:50440
$.addAll$1$ax@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:123996
.Config.setAll$1@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:37466
.Config_Config$fromHtmlContainer@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:37483
.main@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:98655
.main0@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:32528
._IsolateContext.eval$1@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:28202
.startRootIsolate@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:29368
@http://127.0.0.1:3030/.../web/out/index.html_bootstrap.dart.js:131827

What version of the product are you using? On what operating system?
Firefox 22.0 Mac

Please provide any additional information below.
Running document.body.attributes in FF returns [object MozNamedAttrMap].

@jmesserly
Copy link

Set owner to @blois.
Added Area-HTML, Triaged labels.

@jmesserly
Copy link

cc @blois.
Removed the owner.

@jmesserly
Copy link

fwiw, I wasn't able to reproduce this on dart --version 0.6.6.0_r25119. I wonder if somehow Dart auto-update has failed and somehow your html_dart2js.dart file in the SDK is an old version? It should contain the string "MozNamedAttrMap" and likewise the output JS should have this string...


Added NeedsInfo label.

@danschultz
Copy link
Author

I'm unable to repro now. I'm not quite clear what's changed to have fixed this.

@jmesserly
Copy link

Great news! I wonder if it was some strange bug where the SDK didn't quite update or somehow build.dart was failing to rebuild the app & regenerate JS. Glad to hear it's working!


Added AssumedStale label.

@danschultz danschultz added Type-Defect web-libraries Issues impacting dart:html, etc., libraries closed-obsolete Closed as the reported issue is no longer relevant labels Jul 18, 2013
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-obsolete Closed as the reported issue is no longer relevant web-libraries Issues impacting dart:html, etc., libraries
Projects
None yet
Development

No branches or pull requests

3 participants