Oilpan: Prepare moving AXObject to heap

Source/core/dom/Element.h

Index: Source/core/dom/Element.h
diff --git a/Source/core/dom/Element.h b/Source/core/dom/Element.h
index 221c8aeec54020548e849dbeca693885cb853535..eb19f0eb3c7aa737f126845688b28380d1bfe18c 100644
--- a/Source/core/dom/Element.h
+++ b/Source/core/dom/Element.h
@@ -39,6 +39,10 @@
 #include "platform/heap/Handle.h"
 #include "public/platform/WebFocusType.h"
 
+#if ENABLE(OILPAN)
+#include "core/dom/AXObjectCache.h"
+#endif
+
 namespace blink {
 
 class ElementAnimations;
@@ -801,6 +805,10 @@ inline void Node::removedFrom(ContainerNode* insertionPoint)
         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()