Oilpan: Prepare moving AXObject to heap

Source/core/dom/Element.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Peter Kelly (pmk@post.com)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003-2011, 2013, 2014 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 21 matching lines @@
 #include "core/css/CSSSelector.h"
 #include "core/dom/Attribute.h"
 #include "core/dom/ContainerNode.h"
 #include "core/dom/Document.h"
 #include "core/dom/ElementData.h"
 #include "core/dom/SpaceSplitString.h"
 #include "core/html/CollectionType.h"
 #include "platform/heap/Handle.h"
 #include "public/platform/WebFocusType.h"
 
+#if ENABLE(OILPAN)
+#include "core/dom/AXObjectCache.h"
+#endif
+
 namespace blink {
 
 class ElementAnimations;
 class Attr;
 class Attribute;
 class CSSStyleDeclaration;
 class ClientRect;
 class ClientRectList;
 class CustomElementDefinition;
 class DOMStringMap;
@@ skipping 742 matching lines @@
     return InsertionDone;
 }
 
 inline void Node::removedFrom(ContainerNode* insertionPoint)
 {
     ASSERT(insertionPoint->inDocument() || isContainerNode());
     if (insertionPoint->inDocument())
         clearFlag(InDocumentFlag);
     if (isInShadowTree() && !treeScope().rootNode().isShadowRoot())
         clearFlag(IsInShadowTreeFlag);
+#if ENABLE(OILPAN)
+    if (AXObjectCache* cache = document().existingAXObjectCache())
+        cache->remove(this);

[haraken, 2015/06/23 06:33:29]

Can we remove cache->remove() from TreeScopeAdopter::moveTreeToNewScope and
enable this code on both oilpan and non-oilpan? It's not nice that we have a
different lifetime management between oilpan and non-oilpan.

[haraken, 2015/06/23 06:47:22]

Or, in oilpan, isn't the cache->remove() in TreeScopeAdopter::moveTreeToNewScope
enough?

[keishi, 2015/06/24 05:08:21]

Removed excess remove() calls. Node::removeFrom and Document::detach() I think
is enough.

+#endif
 }
 
 inline void Element::invalidateStyleAttribute()
 {
     ASSERT(elementData());
     elementData()->m_styleAttributeIsDirty = true;
 }
 
 inline const StylePropertySet* Element::presentationAttributeStyle()
 {
@@ skipping 63 matching lines @@
     static PassRefPtrWillBeRawPtr<T> create(const QualifiedName&, Document&)
 #define DEFINE_ELEMENT_FACTORY_WITH_TAGNAME(T) \
     PassRefPtrWillBeRawPtr<T> T::create(const QualifiedName& tagName, Document& document) \
     { \
         return adoptRefWillBeNoop(new T(tagName, document)); \
     }
 
 } // namespace
 
 #endif // Element_h