I think I found a temporary fix for the problem...just
calling .datepicker('destroy') on the unload event works for me:
$(window).bind("unload", function() {
$('.hasDatepicker').datepicker('destroy');
$(window).unbind();
});
- Pob
On Jul 3, 1:48 pm, pob <[email protected]> wrote:
> I care about this problem too, and can't find a way of fixing it.
> Please somebody fix it!!!! For the moment I'll go to Kevin Lucks
> version...doesn't look as nice but doesn't leak either!!
>
> On Jul 1, 7:25 pm, "[email protected]"
>
> <[email protected]> wrote:
> > Anybody else care about the above issues?
>
> > On Jun 30, 6:55 pm, "[email protected]"
>
> > <[email protected]> wrote:
> > > It happened using ie also and monitoring in task manager/perfmon
>
> > > On Jun 30, 5:41 pm, mharen <[email protected]> wrote:
>
> > > > I'm not sure drip is a good tool for this job. This page:
>
> > > >http://jsbin.com/eyuri
>
> > > > Shows a leak, too, even though I don't even have a ready() event. All
> > > > it does is include jq/jq-ui.
>
> > > >http://www.picvault.info/images/537090310_eyuri.png
>
> > > > Reloading it in IE6 does *not* show a leak in *this* case. The other
> > > > tests are still leaking.
>
> > > > On Jun 30, 11:59 am, mharen <[email protected]> wrote:
>
> > > > > I'm glad that you're able to reproduce some of the behavior (edited or
> > > > > otherwise)--I'm not going crazy!
>
> > > > > I've read that removing elements makes the problem worse and that
> > > > > setting the .html to "" is better. I think .empty() does that. I
> > > > > wonder what happens if you try it with .empty()?
>
> > > > > On Jun 30, 11:53 am, "[email protected]"
>
> > > > > <[email protected]> wrote:
> > > > > > I have maybe spotted another issue but more jquery related.
> > > > > > Whilst playing with your demo page i tried the following, basically
> > > > > > only adding the remove to the datepicker before reloading.
>
> > > > > > function CheckForRefresh(){
> > > > > >
> > > > > > if($('#AutoRefresh').attr('checked')){
> > > > > >
> > > > > > $('.DatePicker:enabled').remove
> > > > > > ();
> > > > > > window.location.reload();
> > > > > > }
> > > > > > }
>
> > > > > > Look at the drip results for this.....very
> > > > > > worryinghttp://gyazo.com/86d71136e950d1e88e953b8d1e5b71ed.png
>
> > > > > > On Jun 30, 3:53 pm, mharen <[email protected]> wrote:
>
> > > > > > > You're using sp3--which seems to be fixed. Can anyone verify this
> > > > > > > problem with IE6-sans-sp or perhaps with sp1?
>
> > > > > > > Here's drip on my machine, using a now-public
> > > > > > > pagehttp://jsbin.com/omoya
>
> > > > > > >http://www.picvault.info/images/537090308_omoya.png
>
> > > > > > > On Jun 30, 10:12 am, "[email protected]"
>
> > > > > > > <[email protected]> wrote:
> > > > > > > > Sorry should have included that info also, see
> > > > > > > > herehttp://gyazo.com/e4ec10e5ae13df428349c1b8f567214d.png.
>
> > > > > > > > Cheers
>
> > > > > > > > On Jun 30, 2:54 pm, mharen <[email protected]> wrote:
>
> > > > > > > > > What version of IE6 are you using? I've heard reports that
> > > > > > > > > this has
> > > > > > > > > been fixed in IEsp2 and possibly IEsp1. This is repeatable
> > > > > > > > > for me in
> > > > > > > > > IE-sans-sp and IEsp1.
>
> > > > > > > > >http://stackoverflow.com/questions/1051090/how-can-i-control-ie6jquer...
>
> > > > > > > > > Thanks,
> > > > > > > > > Michael Haren
>
> > > > > > > > > On Jun 30, 8:01 am, "[email protected]"
>
> > > > > > > > > <[email protected]> wrote:
> > > > > > > > > > I am not able to reproduce this, see the following drip leak
> > > > > > > > > > screengrab.http://gyazo.com/579a64dc877016ae6d079228fc863728.png
>
> > > > > > > > > > Although memory increases quickly to begin with the gc
> > > > > > > > > > seems to kick
> > > > > > > > > > in at regular intervals leaving a relatively stable
> > > > > > > > > > footprint. That
> > > > > > > > > > seems fine to me considering the amount of reloads that
> > > > > > > > > > occured during
> > > > > > > > > > the test will be way and above over any normal site usage.
>
> > > > > > > > > > On Jun 29, 9:10 pm, mharen <[email protected]> wrote:
>
> > > > > > > > > > > I filed a ticket for this with the repro as a file
> > > > > > > > > > > attachment (http://
> > > > > > > > > > > dev.jqueryui.com/ticket/4644). Sorry if posting here and
> > > > > > > > > > > there is a no-
> > > > > > > > > > > no.
>
> > > > > > > > > > > On Jun 29, 3:35 pm, mharen <[email protected]> wrote:
>
> > > > > > > > > > > > First off: thanks for jquery. I love it. Second: I'm
> > > > > > > > > > > > running into an
> > > > > > > > > > > > IE6 memory leak. Here's a repro:
>
> > > > > > > > > > > > ----begin----
> > > > > > > > > > > > <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0
> > > > > > > > > > > > Transitional//EN"
> > > > > > > > > > > > "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd";>
> > > > > > > > > > > > <html xmlns="http://www.w3.org/1999/xhtml"; >
> > > > > > > > > > > > <head>
> > > > > > > > > > > > <title>Leak Test 2</title>
> > > > > > > > > > > > <link type="text/css"
> > > > > > > > > > > > href="http://ajax.googleapis.com/ajax/libs/
> > > > > > > > > > > > jqueryui/1.7.2/themes/smoothness/jquery-ui.css"
> > > > > > > > > > > > rel="stylesheet" />
> > > > > > > > > > > > <script type="text/javascript"
> > > > > > > > > > > > src="http://ajax.googleapis.com/ajax/
> > > > > > > > > > > > libs/jquery/1.3.2/jquery.min.js"></script>
> > > > > > > > > > > > <script type="text/javascript"
> > > > > > > > > > > > src="http://ajax.googleapis.com/ajax/
> > > > > > > > > > > > libs/jqueryui/1.7.2/jquery-ui.min.js"></script>
>
> > > > > > > > > > > > <script type="text/javascript">
> > > > > > > > > > > > $(document).ready(function() {
> > > > > > > > > > > >
> > > > > > > > > > > > $('.DatePicker:enabled').datepicker();
> > > > > > > > > > > >
> > > > > > > > > > > > setTimeout("CheckForRefresh();",1000);
> > > > > > > > > > > >
> > > > > > > > > > > > $('#AutoRefresh').bind('click',function(){
> > > > > > > > > > > > CheckForRefresh(); });
> > > > > > > > > > > > });
>
> > > > > > > > > > > > function CheckForRefresh(){
> > > > > > > > > > > >
> > > > > > > > > > > > if($('#AutoRefresh').attr('checked')){
> > > > > > > > > > > >
> > > > > > > > > > > > window.location.reload();
> > > > > > > > > > > > }
> > > > > > > > > > > > }
> > > > > > > > > > > > </script>
>
> > > > > > > > > > > > </head>
> > > > > > > > > > > > <body>
> > > > > > > > > > > > <form id='Form1' action='LeakTest2.htm'>
> > > > > > > > > > > > <input class='DatePicker' />
> > > > > > > > > > > > <input id='AutoRefresh'
> > > > > > > > > > > > type='checkbox' checked='checked' /><label
> > > > > > > > > > > > for'AutoRefresh'>Refresh</
> > > > > > > > > > > > label>
> > > > > > > > > > > > </form>
> > > > > > > > > > > > </body>
> > > > > > > > > > > > </html>
> > > > > > > > > > > > ----end----
>
> > > > > > > > > > > > I've tried many things as documented in my
> > > > > > > > > > > > stackoverflow question (see
> > > > > > > > > > > > that for a slightly shorter repro as well). I really
> > > > > > > > > > > > feel like I'm
> > > > > > > > > > > > just throwing spaghetti at the wall to see what sticks
> > > > > > > > > > > > at this point,
> > > > > > > > > > > > though.
>
> > > > > > > > > > > > I tried the nightlies from a few days ago but the
> > > > > > > > > > > > problem persists.
> > > > > > > > > > > > Any help or direction would be greatly appreciated (no,
> > > > > > > > > > > > I can't ignore
> > > > > > > > > > > > IE6 users...they're 3/4 of my corporate users!). Thanks!
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"jQuery Development" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/jquery-dev?hl=en
-~----------~----~----~----~------~----~------~--~---
