Oilpan: move context menu providers to the heap.

Source/core/inspector/InspectorFrontendHost.cpp

 /*
  * Copyright (C) 2007, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Matt Lilek <webkit@mattlilek.com>
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 41 matching lines @@
 #include "platform/SharedBuffer.h"
 #include "platform/UserGestureIndicator.h"
 #include "platform/network/ResourceError.h"
 #include "platform/network/ResourceRequest.h"
 #include "platform/network/ResourceResponse.h"
 
 namespace blink {
 
 class FrontendMenuProvider final : public ContextMenuProvider {
 public:
-    static PassRefPtr<FrontendMenuProvider> create(InspectorFrontendHost* frontendHost, ScriptValue frontendApiObject, const Vector<ContextMenuItem>& items)
+    static PassRefPtrWillBeRawPtr<FrontendMenuProvider> create(InspectorFrontendHost* frontendHost, ScriptValue frontendApiObject, const Vector<ContextMenuItem>& items)
     {
-        return adoptRef(new FrontendMenuProvider(frontendHost, frontendApiObject, items));
+        return adoptRefWillBeNoop(new FrontendMenuProvider(frontendHost, frontendApiObject, items));
+    }
+
+    virtual ~FrontendMenuProvider()
+    {
+        // Verify that this menu provider has been detached.
+        ASSERT(!m_frontendHost);
+    }
+
+    virtual void trace(Visitor* visitor) override
+    {
+        visitor->trace(m_frontendHost);
+        ContextMenuProvider::trace(visitor);
     }
 
     void disconnect()
     {
         m_frontendApiObject = ScriptValue();
-        m_frontendHost = 0;
+        m_frontendHost = nullptr;
     }
 
-private:
-    FrontendMenuProvider(InspectorFrontendHost* frontendHost, ScriptValue frontendApiObject, const Vector<ContextMenuItem>& items)
-        : m_frontendHost(frontendHost)
-        , m_frontendApiObject(frontendApiObject)
-        , m_items(items)
+    virtual void contextMenuCleared() override
     {
-    }
+        if (m_frontendHost) {
+            ScriptFunctionCall function(m_frontendApiObject, "contextMenuCleared");
+            function.call();
 
-    virtual ~FrontendMenuProvider()
-    {
-        contextMenuCleared();
+            m_frontendHost->clearMenuProvider();
+            m_frontendHost = nullptr;
+        }
+        m_items.clear();
     }
 
     virtual void populateContextMenu(ContextMenu* menu) override
     {
         for (size_t i = 0; i < m_items.size(); ++i)
             menu->appendItem(m_items[i]);
     }
 
     virtual void contextMenuItemSelected(const ContextMenuItem* item) override
     {
-        if (m_frontendHost) {
-            UserGestureIndicator gestureIndicator(DefinitelyProcessingNewUserGesture);
-            int itemNumber = item->action() - ContextMenuItemBaseCustomTag;
+        if (!m_frontendHost)
+            return;
 
-            ScriptFunctionCall function(m_frontendApiObject, "contextMenuItemSelected");
-            function.appendArgument(itemNumber);
-            function.call();
-        }
+        UserGestureIndicator gestureIndicator(DefinitelyProcessingNewUserGesture);
+        int itemNumber = item->action() - ContextMenuItemBaseCustomTag;
+
+        ScriptFunctionCall function(m_frontendApiObject, "contextMenuItemSelected");
+        function.appendArgument(itemNumber);
+        function.call();
     }
 
-    virtual void contextMenuCleared() override
+private:
+    FrontendMenuProvider(InspectorFrontendHost* frontendHost, ScriptValue frontendApiObject, const Vector<ContextMenuItem>& items)
+        : m_frontendHost(frontendHost)
+        , m_frontendApiObject(frontendApiObject)
+        , m_items(items)
     {
-        if (m_frontendHost) {
-            ScriptFunctionCall function(m_frontendApiObject, "contextMenuCleared");
-            function.call();
-
-            m_frontendHost->m_menuProvider = 0;
-        }
-        m_items.clear();
-        m_frontendHost = 0;
     }
 
-    InspectorFrontendHost* m_frontendHost;
+    RawPtrWillBeMember<InspectorFrontendHost> m_frontendHost;
     ScriptValue m_frontendApiObject;
+
+    // FIXME: Oilpan: until http://crrev.com/666473003/ is used by the Blink
+    // GC plugin, turn off checking for GC roots for this field. ContextMenuItem
+    // triggers looping behavior.
+    GC_PLUGIN_IGNORE("")

[sof, 2014/10/19 19:36:55]

Will update to refer to http://crbug.com/424962 before finalizing (if the GC
plugin hasn't been updated by then.)

     Vector<ContextMenuItem> m_items;
 };
 
 InspectorFrontendHost::InspectorFrontendHost(InspectorFrontendClient* client, Page* frontendPage)
     : m_client(client)
     , m_frontendPage(frontendPage)
-    , m_menuProvider(0)
+    , m_menuProvider(nullptr)
 {
 }
 
 InspectorFrontendHost::~InspectorFrontendHost()
 {
     ASSERT(!m_client);
 }
 
 void InspectorFrontendHost::trace(Visitor* visitor)
 {
     visitor->trace(m_frontendPage);
+    visitor->trace(m_menuProvider);
 }
 
 void InspectorFrontendHost::disconnectClient()
 {
     m_client = 0;
-    if (m_menuProvider)
+    if (m_menuProvider) {
         m_menuProvider->disconnect();
+        m_menuProvider = nullptr;
+    }
     m_frontendPage = nullptr;
 }
 
 void InspectorFrontendHost::setZoomFactor(float zoom)
 {
     if (!m_frontendPage)
         return;
     if (LocalFrame* frame = m_frontendPage->deprecatedLocalMainFrame())
         frame->setPageAndTextZoomFactors(zoom, 1);
 }
@@ skipping 56 matching lines @@
         m_client->sendMessageToEmbedder(escapeUnicodeNonCharacters(message));
 }
 
 void InspectorFrontendHost::showContextMenu(Page* page, float x, float y, const Vector<ContextMenuItem>& items)
 {
     ASSERT(m_frontendPage);
     ScriptState* frontendScriptState = ScriptState::forMainWorld(m_frontendPage->deprecatedLocalMainFrame());
     ScriptValue frontendApiObject = frontendScriptState->getFromGlobalObject("InspectorFrontendAPI");
     ASSERT(frontendApiObject.isObject());
 
-    RefPtr<FrontendMenuProvider> menuProvider = FrontendMenuProvider::create(this, frontendApiObject, items);
+    RefPtrWillBeRawPtr<FrontendMenuProvider> menuProvider = FrontendMenuProvider::create(this, frontendApiObject, items);
     m_menuProvider = menuProvider.get();
     float zoom = page->deprecatedLocalMainFrame()->pageZoomFactor();
     page->inspectorController().showContextMenu(x * zoom, y * zoom, menuProvider);
 }
 
 void InspectorFrontendHost::showContextMenu(Event* event, const Vector<ContextMenuItem>& items)
 {
     if (!event)
         return;
 
     ASSERT(m_frontendPage);
     ScriptState* frontendScriptState = ScriptState::forMainWorld(m_frontendPage->deprecatedLocalMainFrame());
     ScriptValue frontendApiObject = frontendScriptState->getFromGlobalObject("InspectorFrontendAPI");
     ASSERT(frontendApiObject.isObject());
 
     Page* targetPage = m_frontendPage;
     if (event->target() && event->target()->executionContext() && event->target()->executionContext()->executingWindow()) {
         LocalDOMWindow* window = event->target()->executionContext()->executingWindow();
         if (window->document() && window->document()->page())
             targetPage = window->document()->page();
     }
 
-    RefPtr<FrontendMenuProvider> menuProvider = FrontendMenuProvider::create(this, frontendApiObject, items);
+    RefPtrWillBeRawPtr<FrontendMenuProvider> menuProvider = FrontendMenuProvider::create(this, frontendApiObject, items);
     targetPage->contextMenuController().showContextMenu(event, menuProvider);
     m_menuProvider = menuProvider.get();
 }
 
 String InspectorFrontendHost::getSelectionBackgroundColor()
 {
     return RenderTheme::theme().activeSelectionBackgroundColor().serialized();
 }
 
 String InspectorFrontendHost::getSelectionForegroundColor()
 {
     return RenderTheme::theme().activeSelectionForegroundColor().serialized();
 }
 
 bool InspectorFrontendHost::isUnderTest()
 {
     return m_client && m_client->isUnderTest();
 }
 
 bool InspectorFrontendHost::isHostedMode()
 {
     return false;
 }
 
 } // namespace blink