Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(725)

Unified Diff: Source/core/dom/NthIndexCache.h

Issue 1023393002: Cache element indices for :nth-child and :nth-last-child selectors. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Missing initializer and incorrect assert. Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/dom/Document.cpp ('k') | Source/core/dom/NthIndexCache.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/dom/NthIndexCache.h
diff --git a/Source/core/dom/NthIndexCache.h b/Source/core/dom/NthIndexCache.h
new file mode 100644
index 0000000000000000000000000000000000000000..94f715957f73fd388bf8c6d8ac40ea0760523251
--- /dev/null
+++ b/Source/core/dom/NthIndexCache.h
@@ -0,0 +1,91 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef NthIndexCache_h
+#define NthIndexCache_h
+
+#include "core/dom/Element.h"
+#include "core/dom/ElementTraversal.h"
+#include "platform/heap/Handle.h"
+#include "wtf/HashMap.h"
+#include "wtf/OwnPtr.h"
+#include "wtf/RefPtr.h"
+
+namespace blink {
+
+class Document;
+
+class NthIndexCache final {
+ STACK_ALLOCATED();
+ WTF_MAKE_NONCOPYABLE(NthIndexCache);
+public:
+ explicit NthIndexCache(Document&);
+ ~NthIndexCache();
+
+ inline unsigned nthChildIndex(Element&);
+ inline unsigned nthLastChildIndex(Element&);
+
+private:
+ class NthIndexData final : public NoBaseWillBeGarbageCollected<NthIndexData> {
+ WTF_MAKE_NONCOPYABLE(NthIndexData);
+ DECLARE_EMPTY_DESTRUCTOR_WILL_BE_REMOVED(NthIndexData);
+ public:
+ NthIndexData() { }
+
+ inline unsigned nthIndex(Element&);
+ inline unsigned nthLastIndex(Element&);
+
+ unsigned cacheNthIndices(Element&);
+
+ WillBeHeapHashMap<RawPtrWillBeMember<Element>, unsigned> m_elementIndexMap;
+ unsigned m_count = 0;
+
+ DECLARE_TRACE();
+ };
+
+ NthIndexData& ensureNthIndexDataFor(Node&);
+ inline unsigned nthIndex(Element&);
+
+ using ParentMap = WillBeHeapHashMap<RefPtrWillBeMember<Node>, OwnPtrWillBeMember<NthIndexData>>;
+
+ OwnPtrWillBeMember<ParentMap> m_parentMap;
+ RawPtrWillBeMember<Document> m_document;
+ uint64_t m_domTreeVersion;
+};
+
+inline unsigned NthIndexCache::NthIndexData::nthIndex(Element& element)
+{
+ if (!m_count)
+ return cacheNthIndices(element);
+
+ unsigned index = 0;
+ for (Element* sibling = &element; sibling; sibling = ElementTraversal::previousSibling(*sibling), index++) {
+ auto it = m_elementIndexMap.find(sibling);
+ if (it != m_elementIndexMap.end())
+ return it->value + index;
+ }
+ return index;
+}
+
+inline unsigned NthIndexCache::NthIndexData::nthLastIndex(Element& element)
+{
+ unsigned index = nthIndex(element);
+ return m_count - index + 1;
+}
+
+inline unsigned NthIndexCache::nthChildIndex(Element& element)
+{
+ ASSERT(element.parentNode());
+ return ensureNthIndexDataFor(*element.parentNode()).nthIndex(element);
+}
+
+inline unsigned NthIndexCache::nthLastChildIndex(Element& element)
+{
+ ASSERT(element.parentNode());
+ return ensureNthIndexDataFor(*element.parentNode()).nthLastIndex(element);
+}
+
+} // namespace blink
+
+#endif // NthIndexCache_h
« no previous file with comments | « Source/core/dom/Document.cpp ('k') | Source/core/dom/NthIndexCache.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698