Uploaded image for project: 'Tapestry 5'
  1. Tapestry 5
  2. TAP5-1147

Ajax updates under IE leak lots of memory

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Critical
        • Resolution: Fixed
        • Affects Version/s: 5.2.0
        • Fix Version/s: 5.2.0
        • Component/s: tapestry-core
        • Labels:

          Description

          When an Ajax update changes content on a page, IE wil leak the old DOM nodes under many circumstances. If the DOM nodes have event handlers that are closures that hold a reference back to the DOM node (very easy to do, even without meaning to) a leak will ensue.

            Activity

            Hide
            Permalink
            hlship Howard M. Lewis Ship added a comment -

            Notes on it:

            http://www.outofhanwell.com/ieleak/index.php?title=Main_Page

            http://javascript.crockford.com/memory/leak.html

            this problem has been around for a while.

            The right course of action is to be careful about anything deleted from the DOM. It is necessary to delete any attributes that look like functions (i.e., onclick or onevent attributes) as well as use Prototype's APIs to delete any event handler functions. Only then do the memory leaks disappear.

            Fortunately, I had some time to work through this for a client.

            Show
            hlship Howard M. Lewis Ship added a comment - Notes on it: http://www.outofhanwell.com/ieleak/index.php?title=Main_Page http://javascript.crockford.com/memory/leak.html this problem has been around for a while. The right course of action is to be careful about anything deleted from the DOM. It is necessary to delete any attributes that look like functions (i.e., onclick or onevent attributes) as well as use Prototype's APIs to delete any event handler functions. Only then do the memory leaks disappear. Fortunately, I had some time to work through this for a client.

              People

              • Assignee:
                hlship Howard M. Lewis Ship
                Reporter:
                hlship Howard M. Lewis Ship
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development