Have SelectorQuery API take rootNode by reference

Source/core/css/SelectorCheckerFastPath.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
@@ skipping 21 matching lines @@
 #include "HTMLNames.h"
 #include "core/dom/Element.h"
 #include "core/html/HTMLDocument.h"
 
 namespace WebCore {
 
 using namespace HTMLNames;
 
 namespace {
 
-template <bool checkValue(const Element*, const CSSSelector*)>
+template <bool checkValue(const Element&, const CSSSelector*)>
 inline bool fastCheckSingleSelector(const CSSSelector*& selector, const Element*& element, const CSSSelector*& topChildOrSubselector, const Element*& topChildOrSubselectorMatchElement)
 {
     for (; element; element = element->parentElement()) {
-        if (checkValue(element, selector)) {
+        if (checkValue(*element, selector)) {
             if (selector->relation() == CSSSelector::Descendant)
                 topChildOrSubselector = 0;
             else if (!topChildOrSubselector) {
                 ASSERT(selector->relation() == CSSSelector::Child || selector->relation() == CSSSelector::SubSelector);
                 topChildOrSubselector = selector;
                 topChildOrSubselectorMatchElement = element;
             }
             if (selector->relation() != CSSSelector::SubSelector)
                 element = element->parentElement();
             selector = selector->tagHistory();
             return true;
         }
         if (topChildOrSubselector) {
             // Child or subselector check failed.
             // If the match element is null, topChildOrSubselector was also the very topmost selector and had to match
             // the original element we were checking.
             if (!topChildOrSubselectorMatchElement)
                 return false;
             // There may be other matches down the ancestor chain.
             // Rewind to the topmost child or subselector and the element it matched, continue checking ancestors.
             selector = topChildOrSubselector;
             element = topChildOrSubselectorMatchElement->parentElement();
             topChildOrSubselector = 0;
             return true;
         }
     }
     return false;
 }
 
-inline bool checkClassValue(const Element* element, const CSSSelector* selector)
+inline bool checkClassValue(const Element& element, const CSSSelector* selector)
 {
-    return element->hasClass() && element->classNames().contains(selector->value().impl());
+    return element.hasClass() && element.classNames().contains(selector->value().impl());
 }
 
-inline bool checkIDValue(const Element* element, const CSSSelector* selector)
+inline bool checkIDValue(const Element& element, const CSSSelector* selector)
 {
-    return element->hasID() && element->idForStyleResolution().impl() == selector->value().impl();
+    return element.hasID() && element.idForStyleResolution().impl() == selector->value().impl();
 }
 
-inline bool checkExactAttributeValue(const Element* element, const CSSSelector* selector)
+inline bool checkExactAttributeValue(const Element& element, const CSSSelector* selector)
 {
     return SelectorChecker::checkExactAttribute(element, selector->attribute(), selector->value().impl());
 }
 
-inline bool checkTagValue(const Element* element, const CSSSelector* selector)
+inline bool checkTagValue(const Element& element, const CSSSelector* selector)
 {
     return SelectorChecker::tagMatches(element, selector->tagQName());
 }
 
 }
 
-SelectorCheckerFastPath::SelectorCheckerFastPath(const CSSSelector* selector, const Element* element)
+SelectorCheckerFastPath::SelectorCheckerFastPath(const CSSSelector* selector, const Element& element)
     : m_selector(selector)
     , m_element(element)
 {
 }
 
 bool SelectorCheckerFastPath::matchesRightmostSelector(SelectorChecker::VisitedMatchType visitedMatchType) const
 {
     ASSERT(SelectorCheckerFastPath::canUse(m_selector));
 
     switch (m_selector->m_match) {
@@ skipping 11 matching lines @@
     default:
         ASSERT_NOT_REACHED();
     }
     return false;
 }
 
 bool SelectorCheckerFastPath::matches() const
 {
     ASSERT(matchesRightmostSelector(SelectorChecker::VisitedMatchEnabled));
     const CSSSelector* selector = m_selector;
-    const Element* element = m_element;
+    const Element* element = &m_element;
 
     const CSSSelector* topChildOrSubselector = 0;
     const Element* topChildOrSubselectorMatchElement = 0;
     if (selector->relation() == CSSSelector::Child || selector->relation() == CSSSelector::SubSelector)
         topChildOrSubselector = selector;
 
     if (selector->relation() != CSSSelector::SubSelector)
         element = element->parentElement();
 
     selector = selector->tagHistory();
@@ skipping 61 matching lines @@
     }
     return true;
 }
 
 bool SelectorCheckerFastPath::commonPseudoClassSelectorMatches(SelectorChecker::VisitedMatchType visitedMatchType) const
 {
     ASSERT(SelectorChecker::isCommonPseudoClassSelector(m_selector));
     switch (m_selector->pseudoType()) {
     case CSSSelector::PseudoLink:
     case CSSSelector::PseudoAnyLink:
-        return m_element->isLink();
+        return m_element.isLink();
     case CSSSelector::PseudoVisited:
-        return m_element->isLink() && visitedMatchType == SelectorChecker::VisitedMatchEnabled;
+        return m_element.isLink() && visitedMatchType == SelectorChecker::VisitedMatchEnabled;
     case CSSSelector::PseudoFocus:
         return SelectorChecker::matchesFocusPseudoClass(m_element);
     default:
         ASSERT_NOT_REACHED();
     }
     return true;
 }
 
 
 }