Centralize more querySelector logic behind QuerySelectorCache.

third_party/WebKit/Source/core/dom/ContainerNode.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2013 Apple Inc. All rights
  * reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 16 matching lines @@
 #include "core/dom/ChildFrameDisconnector.h"
 #include "core/dom/ChildListMutationScope.h"
 #include "core/dom/ClassCollection.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/NameNodeList.h"
 #include "core/dom/NodeChildRemovalTracker.h"
 #include "core/dom/NodeComputedStyle.h"
 #include "core/dom/NodeRareData.h"
 #include "core/dom/NodeTraversal.h"
-#include "core/dom/NthIndexCache.h"
 #include "core/dom/SelectorQuery.h"
 #include "core/dom/StaticNodeList.h"
 #include "core/dom/StyleChangeReason.h"
 #include "core/dom/StyleEngine.h"
 #include "core/dom/shadow/ElementShadow.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/events/MutationEvent.h"
 #include "core/frame/FrameView.h"
 #include "core/html/HTMLCollection.h"
 #include "core/html/HTMLFrameOwnerElement.h"
@@ skipping 1118 matching lines @@
 
 unsigned ContainerNode::countChildren() const {
   unsigned count = 0;
   for (Node* node = firstChild(); node; node = node->nextSibling())
     count++;
   return count;
 }
 
 Element* ContainerNode::querySelector(const AtomicString& selectors,
                                       ExceptionState& exceptionState) {
-  if (selectors.isEmpty()) {
-    exceptionState.throwDOMException(SyntaxError,
-                                     "The provided selector is empty.");
-    return nullptr;
-  }
-
   SelectorQuery* selectorQuery = document().selectorQueryCache().add(
       selectors, document(), exceptionState);
   if (!selectorQuery)
     return nullptr;
-
-  NthIndexCache nthIndexCache(document());
   return selectorQuery->queryFirst(*this);
 }
 
 StaticElementList* ContainerNode::querySelectorAll(
     const AtomicString& selectors,
     ExceptionState& exceptionState) {
-  if (selectors.isEmpty()) {
-    exceptionState.throwDOMException(SyntaxError,
-                                     "The provided selector is empty.");
-    return nullptr;
-  }
-
   SelectorQuery* selectorQuery = document().selectorQueryCache().add(
       selectors, document(), exceptionState);
   if (!selectorQuery)
     return nullptr;
-
-  NthIndexCache nthIndexCache(document());
   return selectorQuery->queryAll(*this);
 }
 
 static void dispatchChildInsertionEvents(Node& child) {
   if (child.isInShadowTree())
     return;
 
 #if DCHECK_IS_ON()
   DCHECK(!EventDispatchForbiddenScope::isEventDispatchForbidden());
 #endif
@@ skipping 285 matching lines @@
     return true;
 
   if (node->isElementNode() && toElement(node)->shadow())
     return true;
 
   return false;
 }
 #endif
 
 }  // namespace blink