Oilpan: Prepare to move AXObjectCache to the heap

Source/modules/accessibility/InspectorAccessibilityAgent.cpp

 // Copyright 2014 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.
 
 #include "config.h"
 
 #include "modules/accessibility/InspectorAccessibilityAgent.h"
 
 #include "core/dom/AXObjectCache.h"
 #include "core/dom/DOMNodeIds.h"
@@ skipping 407 matching lines @@
 
     InspectorDOMAgent* domAgent = toLocalFrame(mainFrame)->instrumentingAgents()->inspectorDOMAgent();
     if (!domAgent) {
         *errorString = "DOM agent must be enabled";
         return;
     }
     Node* node = domAgent->assertNode(errorString, nodeId);
     if (!node)
         return;
     Document& document = node->document();
+#if ENABLE(OILPAN)

[haraken, 2015/04/28 10:45:21]

Would you elaborate on why we need a different logic between oilpan and
non-oilpan?

     if (!document.axObjectCache())
         return;
     ScopedAXObjectCache cache(document);
     AXObjectCacheImpl* cacheImpl = toAXObjectCacheImpl(cache.get());
+#else
+    AXObjectCacheImpl* cacheImpl = toAXObjectCacheImpl(document.axObjectCache());
+    if (!cacheImpl)
+        return;
+#endif
     AXObject* axObject = cacheImpl->getOrCreate(node);
     if (!axObject)
         return;
 
     RefPtr<TypeBuilder::Array<AXProperty>> properties = TypeBuilder::Array<AXProperty>::create();
     fillLiveRegionProperties(axObject, properties);
     fillGlobalStates(axObject, properties);
     fillWidgetProperties(axObject, properties);
     fillWidgetStates(axObject, properties);
     fillRelationships(axObject, properties);
 
     accessibilityNode = buildObjectForNode(node, axObject, cacheImpl, properties);
 }
 
 DEFINE_TRACE(InspectorAccessibilityAgent)
 {
     visitor->trace(m_page);
     InspectorBaseAgent::trace(visitor);
 }
 
 } // namespace blink