diff options
| author | Misko Hevery | 2012-05-09 19:27:15 -0400 |
|---|---|---|
| committer | Igor Minar | 2012-05-14 21:56:22 -0700 |
| commit | ec1c5dfaee32f9638cedd28bb96bbbecce9d0cf0 (patch) | |
| tree | 13272649b7df2eb8e9f0d629df527983d66c5d57 /test/testabilityPatch.js | |
| parent | 24e7da4f1954dbe2e89f82950ed00292678534fd (diff) | |
| download | angular.js-ec1c5dfaee32f9638cedd28bb96bbbecce9d0cf0.tar.bz2 | |
fix(jqLite): .data()/.bind() memory leak
Since angular attaches scope/injector/controller
into DOM it should clean up after itself. No need
to complain about memory leaks, since they can
only happened on detached DOM. Detached DOM would
only be in tests, since in production the DOM
would be attached to render tree and removal
would automatically clear memory.
Diffstat (limited to 'test/testabilityPatch.js')
| -rw-r--r-- | test/testabilityPatch.js | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/test/testabilityPatch.js b/test/testabilityPatch.js index 12724192..f033dda2 100644 --- a/test/testabilityPatch.js +++ b/test/testabilityPatch.js @@ -42,7 +42,6 @@ afterEach(function() { var count = 0; forEachSorted(jqCache, function(value, key){ count ++; - delete jqCache[key]; forEach(value, function(value, key){ if (value.$element) { dump('LEAK', key, value.$id, sortedHtml(value.$element)); |
