ICEfaces
  1. ICEfaces
  2. ICE-8763

Performance bottleneck while cleaning up event handlers

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.2
          • Fix Version/s: EE-3.2.0.BETA, EE-3.2.0.GA, 3.3
          • Component/s: Bridge
          • Labels:
            None
          • Environment:
            IE8

            Description


            Just before the DOM updates are applied the bridge tries to cleanup the event handlers of all the elements that will be gone once update is applied. The cleanup can take a significant amount of time (> 10 seconds) when the updates have a large number of elements.

              Activity

              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32130 Tue Nov 13 09:28:53 MST 2012 mircea.toma ICE-8763 Modify findAndNotifyUpdatedElements function to use properties on object to define key-value pairs instead of using the Cell object defined in collections.js. This implementation change speeds up the execution since accessing properties on an object is much faster than using the single method dispatch mechanism used by the Cell object.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/javascript/application.js
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32129 Tue Nov 13 09:25:22 MST 2012 mircea.toma ICE-8763 Changed the cleanup of event handlers before the update is applied to run only over forms elements, anchors, iframes and forms since these are the types that usually have callbacks attached.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/javascript/fixjsf.js

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Mircea Toma
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: