search

dart-archive / polymer-dart

Polymer support for Dart
https://pub.dartlang.org/packages/polymer
BSD 3-Clause "New" or "Revised" License
181 stars 33 forks source link

Investigate memory leak #528

Closed jakemac53 closed 9 years ago

jakemac53 commented 9 years ago

Pretty sure we are leaking memory today, although I haven't checked super recently. This is expected in dartium but not dart2js. This could be coming from polymer js itself, so the best first step is probably to build a quick test to see if it can be duplicated there, and if so then file a bug on that repo.

donny-dont commented 9 years ago

Do you have any more info on it? I remember finding a huge memory leak in non-chrome browsers, https://github.com/Polymer/ShadowDOM/pull/512. I had a potentially automated test for that particular issue I passed onto the Polymer team.

jakemac53 commented 9 years ago

I haven't had any time to investigate it yet - I think it happens regardless of the polyfills though.

jakemac53 commented 9 years ago

looks like we are fine actually