Source/core/inspector/InspectorCanvasAgent.h - Issue 307943002: Oilpan: Prepare moving InspectorController and InspectorAgents to oilpan.

Unified Diff: Source/core/inspector/InspectorCanvasAgent.h

Issue 307943002: Oilpan: Prepare moving InspectorController and InspectorAgents to oilpan. (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Created 6 years, 6 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

Index: Source/core/inspector/InspectorCanvasAgent.h

diff --git a/Source/core/inspector/InspectorCanvasAgent.h b/Source/core/inspector/InspectorCanvasAgent.h

index 1e7af0990afdfce7c355f8f7dfc318bdceaf42a9..3c03d7c81baf3434dfb6ae744c1325180a2645d2 100644

--- a/Source/core/inspector/InspectorCanvasAgent.h

+++ b/Source/core/inspector/InspectorCanvasAgent.h

@@ -54,11 +54,12 @@ typedef String ErrorString;

class InspectorCanvasAgent FINAL : public InspectorBaseAgent<InspectorCanvasAgent>, public InspectorBackendDispatcher::CanvasCommandHandler {

public:

- static PassOwnPtr<InspectorCanvasAgent> create(InspectorPageAgent* pageAgent, InjectedScriptManager* injectedScriptManager)

+ static PassOwnPtrWillBeRawPtr<InspectorCanvasAgent> create(InspectorPageAgent* pageAgent, InjectedScriptManager* injectedScriptManager)

{

- return adoptPtr(new InspectorCanvasAgent(pageAgent, injectedScriptManager));

+ return adoptPtrWillBeNoop(new InspectorCanvasAgent(pageAgent, injectedScriptManager));

}

virtual ~InspectorCanvasAgent();

+ virtual void trace(Visitor*) OVERRIDE;

virtual void setFrontend(InspectorFrontend*) OVERRIDE;

virtual void clearFrontend() OVERRIDE;

@@ -96,7 +97,7 @@ private:

bool checkIsEnabled(ErrorString*) const;

ScriptValue notifyRenderingContextWasWrapped(const ScriptValue&);

- InspectorPageAgent* m_pageAgent;

+ RawPtrWillBeMember<InspectorPageAgent> m_pageAgent;

InjectedScriptManager* m_injectedScriptManager;

keishi 2014/06/11 14:10:54 SAFE: InjectedScriptManager is owned by InspectorC

haraken 2014/06/12 05:53:01 In a follow-up, you can move InjectedScriptManager

keishi 2014/06/13 03:37:33 Done.

InspectorFrontend::Canvas* m_frontend;

keishi 2014/06/11 14:10:54 SAFE: InspectorFrontend::Canvas is owned by Inspec

bool m_enabled;