Attachment #8918161 for bug #1406395

Attachment #8918161: P2: Backout changeset 8f77d260780d (bug 1384661 part 2) for bug #1406395

View | Details | Raw Unified | Return to bug 1406395
Collapse All | Expand All

-Line
+Link Here
- Lines 570-646
  nsAttrChildContentList::IndexOf(nsIConte
+Link Here
+}
 //----------------------------------------------------------------------
+NS_IMETHODIMP
+nsParentNodeChildContentList::GetLength(uint32_t* aLength)
+{
+  if (!mIsCacheValid && !ValidateCache()) {
+    *aLength = 0;
+    return NS_OK;
+  }
+  MOZ_ASSERT(mIsCacheValid);
+  *aLength = mCachedChildArray.Length();
+  return NS_OK;
+}
+NS_IMETHODIMP
+nsParentNodeChildContentList::Item(uint32_t aIndex, nsIDOMNode** aReturn)
+{
+  nsINode* node = Item(aIndex);
+  if (!node) {
+    *aReturn = nullptr;
+    return NS_OK;
+  }
+  return CallQueryInterface(node, aReturn);
+}
+nsIContent*
+nsParentNodeChildContentList::Item(uint32_t aIndex)
+{
+  if (!mIsCacheValid && !ValidateCache()) {
+    return nullptr;
+  }
+  MOZ_ASSERT(mIsCacheValid);
+  return mCachedChildArray.SafeElementAt(aIndex, nullptr);
+}
+int32_t
+nsParentNodeChildContentList::IndexOf(nsIContent* aContent)
+{
+  if (!mIsCacheValid && !ValidateCache()) {
+    return -1;
+  }
+  MOZ_ASSERT(mIsCacheValid);
+  return mCachedChildArray.IndexOf(aContent);
+}
+bool
+nsParentNodeChildContentList::ValidateCache()
+{
+  MOZ_ASSERT(!mIsCacheValid);
+  MOZ_ASSERT(mCachedChildArray.IsEmpty());
+  nsINode* parent = GetParentObject();
+  if (!parent) {
+    return false;
+  }
+  for (nsIContent* node = parent->GetFirstChild(); node;
+       node = node->GetNextSibling()) {
+    mCachedChildArray.AppendElement(node);
+  }
+  mIsCacheValid = true;
+  return true;
+}
+//----------------------------------------------------------------------
 nsIHTMLCollection*
 FragmentOrElement::Children()





#define nsChildContentList_h__

#include "nsISupportsImpl.h"
#include "nsINodeList.h"            // base class
#include "js/TypeDecls.h"     // for Handle, Value, JSObject, JSContext

class nsIContent;

 * and Item to its existing child list.
 * @see nsIDOMNodeList
 */
class nsAttrChildContentList final : public nsINodeList
{
public:
  explicit nsAttrChildContentList(nsINode* aNode)

  virtual int32_t IndexOf(nsIContent* aContent) override;
  virtual nsIContent* Item(uint32_t aIndex) override;

  void DropReference()
  {
    mNode = nullptr;
  }

    return mNode;
  }

private:
  ~nsAttrChildContentList() {}

private:
  // The node whose children make up the list.
  // This is a non-owning ref which is safe because it's set to nullptr by
  // DropReference() by the node slots get destroyed.
  nsINode* MOZ_NON_OWNING_REF mNode;
};

class nsParentNodeChildContentList final : public nsAttrChildContentList
{
public:
  explicit nsParentNodeChildContentList(nsINode* aNode)
    : nsAttrChildContentList(aNode)
    , mIsCacheValid(false)
  {
    ValidateCache();
  }

  // nsIDOMNodeList interface
  NS_DECL_NSIDOMNODELIST

  // nsINodeList interface
  virtual int32_t IndexOf(nsIContent* aContent) override;
  virtual nsIContent* Item(uint32_t aIndex) override;

  void DropReference() override
  {
    InvalidateCache();
    nsAttrChildContentList::DropReference();
  }

  void InvalidateCache()
  {
    mIsCacheValid = false;
    mCachedChildArray.Clear();
  }

private:
  ~nsParentNodeChildContentList() {}

  // Return true if validation succeeds, false otherwise
  bool ValidateCache();

  // Whether cached array of child nodes is valid
  bool mIsCacheValid;

  // Cached array of child nodes
  AutoTArray<nsIContent*, 8> mCachedChildArray;
};

#endif /* nsChildContentList_h__ */

Return to bug 1406395

Bugzilla

Quick Search

Attachment #8918161: P2: Backout changeset 8f77d260780d (bug 1384661 part 2) for bug #1406395