Skip to content

Commit

Permalink
Use native qSA
Browse files Browse the repository at this point in the history
Add documentation about when to use scopeSubtree
  • Loading branch information
dfreedm committed May 8, 2019
1 parent 6bc9534 commit e10019a
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions lib/utils/scope-subtree.js
Expand Up @@ -9,13 +9,14 @@ subject to an additional IP rights grant found at http://polymer.github.io/PATEN
*/

import './boot.js';
import { wrap } from './wrap.js';

const ShadyDOM = window.ShadyDOM;
const ShadyCSS = window.ShadyCSS;

/**
* Ensure that elements in a ShadowDOM container are scoped correctly.
* This function is only needed when ShadyDOM is used and unpatched DOM APIs are used in third party code.
* This can happen in noPatch mode or when specialized APIs like ranges or tables are used to mutate DOM.
*
* @param {!Element} container Container element to scope
* @param {boolean=} shouldObserve if true, start a mutation observer for added nodes to the container
Expand All @@ -26,8 +27,8 @@ export function scopeSubtree(container, shouldObserve = false) {
if (!ShadyDOM || !ShadyCSS) {
return null;
}
// ShadyCSS handles DOM mutations in patched mode
if (!ShadyDOM['noPatch']) {
// ShadyCSS handles DOM mutations when ShadyDOM does not handle scoping itself
if (!ShadyDOM['handlesDynamicScoping']) {
return null;
}
const ScopingShim = ShadyCSS['ScopingShim'];
Expand All @@ -39,8 +40,9 @@ export function scopeSubtree(container, shouldObserve = false) {
const containerScope = ScopingShim['scopeForNode'](container);

const scopify = (node) => {
const wrappedNode = wrap(node);
const elements = [node, ...(wrappedNode.querySelectorAll('*'))];
// NOTE: native qSA does not honor scoped DOM, but it is faster, and the same behavior as Polymer v1
const descendants = ShadyDOM['nativeMethods']['querySelectorAll'].call(node, '*');
const elements = [node, ...descendants];
for (let i = 0; i < elements.length; i++) {
const el = elements[i];
const currentScope = ScopingShim['currentScopeForNode'](el);
Expand Down

0 comments on commit e10019a

Please sign in to comment.