jQuery/Sizzle checkContext perdita di memoria

Question

Durante il debug della mia app con "Profili" in DevTools ho trovato "accumulato l'albero DOM" che si accumula. Questi nodi scollegati hanno un albero di contenimento composto principalmente dalle funzioni checkContext (provenienti da sizzle all'interno di jQuery - v1.10.1).

Non sono sicuro di come procedere con questo. Cosa significa questo risultato?

Answer 1

Questo è in realtà un bug, non c'è motivo per cui Sizzle debba rimanere nel nodo di contesto, lo fa solo perché non si sta pulendo dopo aver impostato una variabile temporanea. Ho archiviato uno issue for it, risolto, eseguito tutti i test Sizzle e fatto una richiesta di pull.

Se si vuole correggere la vostra copia esistente di jQuery o Sizzle:

1. Aprire il jQuery o Sizzle file di

2. Cercare il matcherFromTokens funzione di

3. Trova questo codice al suo interno (vicino alla cima):

   matchers = [ function(elem, context, xml) { 
       return (!leadingRelative && (xml || context !== outermostContext)) || (
        (checkContext = context).nodeType ? 
         matchContext(elem, context, xml) : 
         matchAnyContext(elem, context, xml)); 
   } ]; 
   

4. Modificare il return-var rv =, e aggiungere checkContext = undefined; e poi return rv; alla fine della funzione anonima, ad es .:

   matchers = [ function(elem, context, xml) { 
       var ret = (!leadingRelative && (xml || context !== outermostContext)) || (
        (checkContext = context).nodeType ? 
         matchContext(elem, context, xml) : 
         matchAnyContext(elem, context, xml)); 
       // Release the context node (issue #299) 
       checkContext = null; 
       return ret; 
   } ]; 
   

Nota: Questo codice assegna null per checkContext perché a quanto pare questo è il loro stile. Se fossi in me, avrei assegnato lo undefined.

Se c'è qualche problema con la correzione sollevata durante il processo di richiesta/fusione, aggiornerò la risposta.

È meglio continuare a lasciare selettori di cache Sizzle, perché jQuery usa il selettore compilato con la delega degli eventi e non si vuole davvero che debba ripetere e ricostruire le funzioni di matcher ogni volta che si verifica un evento rilevante in modo che possa capire se gli elementi corrispondono.

---

Questo non è l'unico posto che jQuery conserva sugli elementi nei selettori compilati, sfortunatamente. Ogni posto che fa è probabilmente un bug che potrebbe usare il fixing.Ho solo avuto il tempo di rintracciare un altro, che ho anche segnalato e risolto (in attesa della richiesta di pull essere atterrato):

Se si cerca "pseudos potenzialmente complesse" troverete questo per la :not pseudo-selettore:

pseudos: { 
    // Potentially complex pseudos 
    "not": markFunction(function(selector) { 
     // Trim the selector passed to compile 
     // to avoid treating leading and trailing 
     // spaces as combinators 
     var input = [], 
      results = [], 
      matcher = compile(selector.replace(rtrim, "$1")); 

     return matcher[ expando ] ? 
      markFunction(function(seed, matches, context, xml) { 
       var elem, 
        unmatched = matcher(seed, null, xml, []), 
        i = seed.length; 

       // Match elements unmatched by `matcher` 
       while (i--) { 
        if ((elem = unmatched[i])) { 
         seed[i] = !(matches[i] = elem); 
        } 
       } 
      }) : 
      function(elem, context, xml) { 
       input[0] = elem; 
       matcher(input, null, xml, results); 
       return !results.pop(); 
      }; 
    }), 

il problema è in funzione, dopo l': nell'operatore condizionale:

function(elem, context, xml) { 
    input[0] = elem; 
    matcher(input, null, xml, results); 
    return !results.pop(); 
}; 

noti che non cancella input[0]. Ecco la soluzione:

function(elem, context, xml) { 
    input[0] = elem; 
    matcher(input, null, xml, results); 
    // Don't keep the element (issue #299) 
    input[0] = null; 
    return !results.pop(); 
}; 

Questo è tutto quello che ho il tempo di rintracciare al momento.

Answer 2

Sizzle memorizza i selettori nella cache del selettore, che per impostazione predefinita memorizza fino a 50 voci. Potresti sperimentare impostando $.expr.cacheLength = 1 prima di fare qualsiasi selezione e vedere se se ne vanno.

Ecco i documenti https://github.com/jquery/sizzle/wiki/Sizzle-Documentation#-internal-api. Sembra interno quindi non dipendono da esso o altro nel codice di produzione attuale.