Simplify the id fast path for non-right most selectors in querySelector.

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

 /*
  * Copyright (C) 2011, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2014 Samsung Electronics. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 226 matching lines @@
   // We need to return the matches in document order. To use id lookup while
   // there is possiblity of multiple matches we would need to sort the
   // results. For now, just traverse the document in that case.
   DCHECK_EQ(m_selectors.size(), 1u);
 
   bool isRightmostSelector = true;
   bool startFromParent = false;
 
   for (const CSSSelector* selector = m_selectors[0]; selector;
        selector = selector->tagHistory()) {
-    if (selector->match() == CSSSelector::Id && rootNode.isInTreeScope() &&
+    if (selector->match() == CSSSelector::Id &&
         !rootNode.containingTreeScope().containsMultipleElementsWithId(
             selector->value())) {
+      // Id selectors in the right most selector are handled by the caller,
+      // we should never hit them here.
+      DCHECK(!isRightmostSelector);
       Element* element =
           rootNode.containingTreeScope().getElementById(selector->value());
-      ContainerNode* adjustedNode = &rootNode;
-      if (element && element->isDescendantOf(&rootNode))
-        adjustedNode = element;
-      else if (!element || isRightmostSelector)
-        adjustedNode = nullptr;
-      if (isRightmostSelector) {
-        if (!adjustedNode)
-          return;
-        element = toElement(adjustedNode);
-        if (selectorMatches(*m_selectors[0], *element, rootNode))
-          SelectorQueryTrait::appendElement(output, *element);
+      if (!element)
         return;
-      }
-
-      if (startFromParent && adjustedNode)
-        adjustedNode = adjustedNode->parentNode();
-
-      executeForTraverseRoot<SelectorQueryTrait>(adjustedNode, rootNode,
-                                                 output);
+      ContainerNode* start = &rootNode;
+      if (element->isDescendantOf(&rootNode))
+        start = element;
+      if (startFromParent)
+        start = start->parentNode();
+      executeForTraverseRoot<SelectorQueryTrait>(start, rootNode, output);
       return;
     }
 
     // If we have both CSSSelector::Id and CSSSelector::Class at the same time,
     // we should use Id to find traverse root.
     if (!SelectorQueryTrait::shouldOnlyMatchFirstElement && !startFromParent &&
         selector->match() == CSSSelector::Class) {
       if (isRightmostSelector) {
         ClassElementList<AllElements> traverseRoots(rootNode,
                                                     selector->value());
@@ skipping 296 matching lines @@
   return m_entries
       .insert(selectors, SelectorQuery::adopt(std::move(selectorList)))
       .storedValue->value.get();
 }
 
 void SelectorQueryCache::invalidate() {
   m_entries.clear();
 }
 
 }  // namespace blink