ICEfaces
  1. ICEfaces
  2. ICE-5628

Memory leak in IE7 using icefaces2

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Alpha3
          • Fix Version/s: 2.0-Alpha3, 2.0.0
          • Component/s: Bridge
          • Labels:
            None
          • Environment:
            IE7 & Glassfishv3-final

            Description

            Auction application was left running over night on glassfiahv3 final server. Below are the memory readings:

            Day1 morning 20,528
                      Day1 evening 335,260
                      Day2 morning 536,756
                      Minimize 502,192
                      Maximize 523,912


            Memory readings for auctionMonitor are as follows:

                       Day1 morning 45,316
                       Day1 evening 986,012
                             

              Activity

              Ascending order - Click to sort in descending order
              Mandeep Hayher created issue -
              Mandeep Hayher made changes -
              Field Original Value New Value
              Assignee Ken Fyten [ ken.fyten ]
              Mandeep Hayher made changes -
              Salesforce Case []
              Description Auction application was left running over night on glassfiahv3 final server. Below are the memory readings:

              day1 morning day1 evening day2 morning Minimize Maximize
              IE7 20,528 335,260 536,756 502,192 523,912

              		 Auction application was left running over night on glassfiahv3 final server. Below are the memory readings:

              Day1 morning Day1 evening Day2 morning Minimize Maximize
              IE7 20,528 335,260 536,756 502,192 523,912

              Memory readings for auctionMonitor are as follows:

                         Day1 morning Day1 evening
              IE7 45,316 986,012
              Mandeep Hayher made changes -
              Salesforce Case []
              Description Auction application was left running over night on glassfiahv3 final server. Below are the memory readings:

              Day1 morning Day1 evening Day2 morning Minimize Maximize
              IE7 20,528 335,260 536,756 502,192 523,912

              Memory readings for auctionMonitor are as follows:

                         Day1 morning Day1 evening
              IE7 45,316 986,012               		Auction application was left running over night on glassfiahv3 final server. Below are the memory readings:

              Day1 morning 20,528
                        Day1 evening 335,260
                        Day2 morning 536,756
                        Minimize 502,192
                        Maximize 523,912


              Memory readings for auctionMonitor are as follows:

                         Day1 morning 45,316
                         Day1 evening 986,012
                               
              Ken Fyten made changes -
              Salesforce Case []
              Component/s Bridge [ 10011 ]
              Fix Version/s 2.0-Alpha3 [ 10032 ]
              Assignee Ken Fyten [ ken.fyten ] Mircea Toma [ mircea.toma ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #21285 Tue Apr 20 13:38:04 MDT 2010 mircea.toma ICE-5628 Stop cloning the form used for retrieving updates.
              Files Changed
              Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/javascript/application.js
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #21286 Tue Apr 20 14:40:45 MDT 2010 mircea.toma ICE-5628 Stop cloning the form used for retrieving updates.
              Files Changed
              Commit graph MODIFY /icefaces/scratchpads/glimmer/core/src/main/javascript/submit.js
              Hide
              Permalink
              Mircea Toma added a comment - - edited

              It seems that when the form used by retrieveUpdates() and singleSubmit() is cloned, even if not appended to the document, generates memory leaks in IE.
              The solution was to use the form directly by just editing its attributes or append elements, after the request is made the edited values are rolled back and appended elements removed.

              Show
              Mircea Toma added a comment - - edited It seems that when the form used by retrieveUpdates() and singleSubmit() is cloned, even if not appended to the document, generates memory leaks in IE. The solution was to use the form directly by just editing its attributes or append elements, after the request is made the edited values are rolled back and appended elements removed.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              QA reports that the leak still occurs with IE:

              Mandeep says:

              Looks like the memory leak is still present

              Revision# 21290
              auctionMonitor icepc13

              Start End
              IE7 46,156 972,244 (clocks stop ticking)

              Auction tests are starting now, so should have results by end of today.

              Show
              Ken Fyten added a comment - QA reports that the leak still occurs with IE: Mandeep says: Looks like the memory leak is still present Revision# 21290 auctionMonitor icepc13 Start End IE7 46,156 972,244 (clocks stop ticking) Auction tests are starting now, so should have results by end of today.
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P1
              Hide
              Permalink
              Ken Fyten added a comment -

              Reran browser memory test:
              auction ICEPC18

              IE7 31,708 403,236 (clocks stopped ticking)

              auctionMonitor ICEPC13
              IE7 44,064 969,804 (clocks stopped ticking)

              The browser cache was clean.

              Regards,
              Mandeep

              Show
              Ken Fyten added a comment - Reran browser memory test: auction ICEPC18 IE7 31,708 403,236 (clocks stopped ticking) auctionMonitor ICEPC13 IE7 44,064 969,804 (clocks stopped ticking) The browser cache was clean. Regards, Mandeep
              Hide
              Permalink
              Mircea Toma added a comment - - edited

              The Mojarra 2.0.2 libraries don't contain the fix we sent for the memory leak in JSF bridge: https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1521 . The tests should be run with libraries build from code with the revision greater than #8306 (Mojarra repository).

              Show
              Mircea Toma added a comment - - edited The Mojarra 2.0.2 libraries don't contain the fix we sent for the memory leak in JSF bridge: https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1521 . The tests should be run with libraries build from code with the revision greater than #8306 (Mojarra repository).
              Ken Fyten made changes -
              Assignee Mircea Toma [ mircea.toma ] Mandeep Hayher [ mandeep.hayher ]
              Hide
              Permalink
              Ken Fyten added a comment -

              Weekend testing confirms that this leak is fixed. Tested versions are:

              Glimmer revision: 21420
              jsf2.0-mojarra-r8306
              Server: Tomcat 6.0.26

              Show
              Ken Fyten added a comment - Weekend testing confirms that this leak is fixed. Tested versions are: Glimmer revision: 21420 jsf2.0-mojarra-r8306 Server: Tomcat 6.0.26
              Ken Fyten made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Assignee Priority P1
              Resolution Fixed [ 1 ]
              Assignee Mandeep Hayher [ mandeep.hayher ]
              Ken Fyten made changes -
              Fix Version/s 2.0.0 [ 10230 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Mandeep Hayher
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: