Guriddo » Topic: SubGrid options and Memory Leaks

SubGrid options and Memory Leaks

Guriddo
SubGrid options and Memory Leaks

This topic has 9 replies, 5 voices, and was last updated 14 years, 3 months ago by yejinzai.

Creator

Topic
February 28, 2012 at 2:50 pm #79689

caradees
Participant

Hello,

Attached below is a basic jqGrid example – to simulate auto-refresh conditions. I observed that all you need to do is add this option of subGrid (subGrid: true,) in the jqGrid options to start a memory leak. Without this options the iexplore leaks dissapear.

Am I not using the grid options correctly ? Attached is the code for this HTML page. This is observed in IE 8 when the start button is clicked after creating the Grid using the Create click.

I am using jqGrid 4.1.2 and jQuery 1.6.2.

Appreciate your inputs/suggestions on this.

-Caradee

——–

jqGrid Example
Creator

Topic

Viewing 9 replies - 1 through 9 (of 9 total)

Author

Replies
February 29, 2012 at 2:50 pm #99536

Nisha09
Participant

Hello,

Even I am also facing the same issue with jqGrid 4.1.2 and jQuery 1.6.2…In each refresh of grid data(Using timer),

March 1, 2012 at 1:06 pm #99540

tony
Keymaster

Hello,

Maybe you will need to use a certain DOCTYPE.

Using jQuery 1.6.x is not recommended.

Please look at thiew website the problems related to IE.

Regards

March 5, 2012 at 3:15 pm #99550

caradees
Participant

tony said:

Hello,

Maybe you will need to use a certain DOCTYPE.

Using jQuery 1.6.x is not recommended.

Please look at thiew website the problems related to IE.

Regards

Tony,

Thanks for that input. We experimented with various document types and observed the memory usage patterns of jqGrid with the subGrid set to true within IE8. Here is what we found (please note that we used a customized setData method within jqGrid 4.3.1 – will post that function later once we have something concrete)

jQuery version used was still 1.6.2

DT1 –

DT2 –

DT3 –

DT4 –

As you can see from the graph below that setting it to DT3 (Quirks mode) did not cause the leaks as in the other document modes.

Interesting. Will post more as we find with our experiments. Thanks for putting our heads into that direction, Tony !!!

Appreciate your help and inputs. Please do add your thoughts/comments on this.

March 12, 2012 at 2:51 pm #99578

caradees
Participant

Enabling Quirks mode is changing the way the application is being rendered and moving forward since HTML5 might become the standard I would like to not enable quirks mode in IE – but I need to enable the jqGrid subgrid to implement a feature.

Is it possible that the new rows being inserted where the subgrid is present is not being garbage collected by IE.

The mainGrid being

March 15, 2012 at 10:31 am #99600

caradees
Participant

I started to dig into the jqGrid code for addSubGrid in jqGrid 4.1.2 to check what might be causing the Internet Explorer leak when subGrids are enabled. Finally we narrowed down to this line of code that binds the click event to the cell

March 15, 2012 at 11:43 am #99601

caradees
Participant

Regarding my last post,

Please disregard – ts = null; option that breaks the functionality of the jqGrid subGrid (we will now be unable to click on the + sign)

Binding the click event to each of the 10 rows in the jqGrid takes up about 35Kb. Since I have 10 rows per page my implementation my application is leaking by 350Kb per refresh.

I am not sure how to release this memory that is being consumed by this statement

$(ts.rows.cells[pos]).bind('click', function(e) { /* SOME CODE HERE */ });

Somebody please help

March 15, 2012 at 8:32 pm #99603

OlegK
Participant

Hello caradees,

You made many experiments to localize the problem and to find a workaround. One more test you can do if you suspect that

$(ts.rows.cells[pos]).bind('click', function(e) { /* SOME CODE HERE */ });
is the origin of the problem. The code set 'click' event on every '+' icon. The last line of the event handler is return false; which stop event propagation. If you remove (comment) the code you can use
onCellSelect: function (rowid, iCol, cellcontent, e) {

April 25, 2012 at 10:19 am #99765

yejinzai
Participant

Hi Oleg,

I'm kinda confused on what to do based on your comment where $(ts.rows.cells[pos]).bind('click', function(e) { /* SOME CODE HERE */ }); is written on the jquery source file while onCellSelect: function (rowid, iCol, cellcontent, e) { will be written on the client code. Can you englighten me?

I'm creating another grid as subgrid and users experience memory error after a number of continuous expanding. Below is my sample code if needed.

subGrid: true, subGridRowExpanded: function(subgrid_id, row_id) {

April 25, 2012 at 12:58 pm #99754

OlegK
Participant

Hi Kris,

I agree that I expressed me not clear enough in my last post. What I suggest is mostly the code optimization of addSubGrid
Author

Replies