[SCM] WebKit Debian packaging branch, webkit-1.3, updated. upstream/1.3.7-4207-g178b198

loislo at chromium.org loislo at chromium.org
Sun Feb 20 23:25:24 UTC 2011


The following commit has been merged in the webkit-1.3 branch:
commit f2d264d4514db2a66abcd526790b67ab05b5c00e
Author: loislo at chromium.org <loislo at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Jan 20 12:38:16 2011 +0000

    2011-01-20  Ilya Tikhonovsky  <loislo at chromium.org>
    
            Reviewed by Yury Semikhatsky.
    
            Web Inspector: prepare for extracting InspectorAgent from InspectorController.
            https://bugs.webkit.org/show_bug.cgi?id=52797
    
            * inspector/CodeGeneratorInspector.pm:
            * inspector/InspectorController.h:
            (WebCore::InspectorController::inspectorAgent):
            (WebCore::InspectorController::cssAgent):
            (WebCore::InspectorController::injectedScriptAgent):
            (WebCore::InspectorController::resourceAgent):
            (WebCore::InspectorController::runtimeAgent):
            (WebCore::InspectorController::timelineAgent):
            (WebCore::InspectorController::databaseAgent):
            (WebCore::InspectorController::domStorageAgent):
            (WebCore::InspectorController::fileSystemAgent):
            (WebCore::InspectorController::browserDebuggerAgent):
            (WebCore::InspectorController::debuggerAgent):
            (WebCore::InspectorController::profilerAgent):
            (WebCore::InspectorController::applicationCacheAgent):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@76227 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 82470af..fd99753 100644
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,26 @@
+2011-01-20  Ilya Tikhonovsky  <loislo at chromium.org>
+
+        Reviewed by Yury Semikhatsky.
+
+        Web Inspector: prepare for extracting InspectorAgent from InspectorController.
+        https://bugs.webkit.org/show_bug.cgi?id=52797
+
+        * inspector/CodeGeneratorInspector.pm:
+        * inspector/InspectorController.h:
+        (WebCore::InspectorController::inspectorAgent):
+        (WebCore::InspectorController::cssAgent):
+        (WebCore::InspectorController::injectedScriptAgent):
+        (WebCore::InspectorController::resourceAgent):
+        (WebCore::InspectorController::runtimeAgent):
+        (WebCore::InspectorController::timelineAgent):
+        (WebCore::InspectorController::databaseAgent):
+        (WebCore::InspectorController::domStorageAgent):
+        (WebCore::InspectorController::fileSystemAgent):
+        (WebCore::InspectorController::browserDebuggerAgent):
+        (WebCore::InspectorController::debuggerAgent):
+        (WebCore::InspectorController::profilerAgent):
+        (WebCore::InspectorController::applicationCacheAgent):
+
 2011-01-20  Sheriff Bot  <webkit.review.bot at gmail.com>
 
         Unreviewed, rolling out r76215.
diff --git a/Source/WebCore/inspector/CodeGeneratorInspector.pm b/Source/WebCore/inspector/CodeGeneratorInspector.pm
index ea98d09..e215a91 100644
--- a/Source/WebCore/inspector/CodeGeneratorInspector.pm
+++ b/Source/WebCore/inspector/CodeGeneratorInspector.pm
@@ -13,70 +13,70 @@ my %typeTransform;
 $typeTransform{"ApplicationCache"} = {
     "forward" => "InspectorApplicationCacheAgent",
     "header" => "InspectorApplicationCacheAgent.h",
-    "domainAccessor" => "m_inspectorController->m_applicationCacheAgent",
+    "domainAccessor" => "m_inspectorAgent->applicationCacheAgent()",
 };
 $typeTransform{"CSS"} = {
     "forward" => "InspectorCSSAgent",
     "header" => "InspectorCSSAgent.h",
-    "domainAccessor" => "m_inspectorController->m_cssAgent",
+    "domainAccessor" => "m_inspectorAgent->cssAgent()",
 };
 $typeTransform{"Console"} = {
     "forward" => "InspectorConsoleAgent",
     "header" => "InspectorConsoleAgent.h",
-    "domainAccessor" => "m_inspectorController->m_consoleAgent",
+    "domainAccessor" => "m_inspectorAgent->consoleAgent()",
 };
 $typeTransform{"Debugger"} = {
     "forward" => "InspectorDebuggerAgent",
     "header" => "InspectorDebuggerAgent.h",
-    "domainAccessor" => "m_inspectorController->m_debuggerAgent",
+    "domainAccessor" => "m_inspectorAgent->debuggerAgent()",
 };
 $typeTransform{"BrowserDebugger"} = {
     "forward" => "InspectorBrowserDebuggerAgent",
     "header" => "InspectorBrowserDebuggerAgent.h",
-    "domainAccessor" => "m_inspectorController->m_browserDebuggerAgent",
+    "domainAccessor" => "m_inspectorAgent->browserDebuggerAgent()",
 };
 $typeTransform{"Database"} = {
     "forward" => "InspectorDatabaseAgent",
     "header" => "InspectorDatabaseAgent.h",
-    "domainAccessor" => "m_inspectorController->m_databaseAgent",
+    "domainAccessor" => "m_inspectorAgent->databaseAgent()",
 };
 $typeTransform{"DOM"} = {
     "forward" => "InspectorDOMAgent",
     "header" => "InspectorDOMAgent.h",
-    "domainAccessor" => "m_inspectorController->m_domAgent",
+    "domainAccessor" => "m_inspectorAgent->domAgent()",
 };
 $typeTransform{"DOMStorage"} = {
     "forward" => "InspectorDOMStorageAgent",
     "header" => "InspectorDOMStorageAgent.h",
-    "domainAccessor" => "m_inspectorController->m_domStorageAgent",
+    "domainAccessor" => "m_inspectorAgent->domStorageAgent()",
 };
 $typeTransform{"FileSystem"} = {
     "forward" => "InspectorFileSystemAgent",
     "header" => "InspectorFileSystemAgent.h",
-    "domainAccessor" => "m_inspectorController->m_fileSystemAgent",
+    "domainAccessor" => "m_inspectorAgent->fileSystemAgent()",
 };
 $typeTransform{"InjectedScript"} = {
     "forwardHeader" => "InjectedScriptHost.h",
-    "domainAccessor" => "m_inspectorController->m_injectedScriptHost",
+    "domainAccessor" => "m_inspectorAgent->injectedScriptAgent()",
 };
 $typeTransform{"Inspector"} = {
-    "forwardHeader" => "InspectorController.h",
-    "domainAccessor" => "m_inspectorController",
+    "forwardHeader" => "InspectorController.h", # FIXME: Temporary solution until extracting the real InspectorAgent from InspectorController.
+    "domainAccessor" => "m_inspectorAgent",
 };
 $typeTransform{"Network"} = {
     "forward" => "InspectorResourceAgent",
     "header" => "InspectorResourceAgent.h",
-    "domainAccessor" => "m_inspectorController->m_resourceAgent",
+    "domainAccessor" => "m_inspectorAgent->resourceAgent()",
 };
 $typeTransform{"Profiler"} = {
     "forward" => "InspectorProfilerAgent",
     "header" => "InspectorProfilerAgent.h",
-    "domainAccessor" => "m_inspectorController->m_profilerAgent",
+    "domainAccessor" => "m_inspectorAgent->profilerAgent()",
 };
 $typeTransform{"Runtime"} = {
     "forward" => "InspectorRuntimeAgent",
     "header" => "InspectorRuntimeAgent.h",
-    "domainAccessor" => "m_inspectorController->m_runtimeAgent",
+    "domainAccessor" => "m_inspectorAgent->runtimeAgent()",
 };
 
 $typeTransform{"Frontend"} = {
@@ -269,12 +269,13 @@ sub GenerateInterface
     $backendClassName = $className . "BackendDispatcher";
     $backendJSStubName = $className . "BackendStub";
     my @backendHead;
-    push(@backendHead, "    ${backendClassName}(InspectorController* inspectorController) : m_inspectorController(inspectorController) { }");
+    push(@backendHead, "    typedef InspectorController InspectorAgent;"); # FIXME: Temporary substitution until extracting InspectorAgent from InspectorController.
+    push(@backendHead, "    ${backendClassName}(InspectorAgent* inspectorAgent) : m_inspectorAgent(inspectorAgent) { }");
     push(@backendHead, "    void reportProtocolError(const long callId, const String& errorText) const;");
     push(@backendHead, "    void dispatch(const String& message);");
     push(@backendHead, "    static bool getCommandName(const String& message, String* result);");
     $backendConstructor = join("\n", @backendHead);
-    $backendFooter = "    InspectorController* m_inspectorController;";
+    $backendFooter = "    InspectorAgent* m_inspectorAgent;";
     $backendTypes{"Inspector"} = 1;
     $backendTypes{"InspectorClient"} = 1;
     $backendTypes{"PassRefPtr"} = 1;
@@ -411,7 +412,7 @@ sub generateBackendFunction
     }
 
     push(@function, "    // use InspectorFrontend as a marker of WebInspector availability");
-    push(@function, "    if ((callId || protocolErrors->length()) && m_inspectorController->hasFrontend()) {");
+    push(@function, "    if ((callId || protocolErrors->length()) && m_inspectorAgent->hasFrontend()) {");
     push(@function, "        RefPtr<InspectorObject> responseMessage = InspectorObject::create();");
     push(@function, "        responseMessage->setNumber(\"seq\", callId);");
     push(@function, "        responseMessage->setString(\"domain\", \"$domain\");");
@@ -426,7 +427,7 @@ sub generateBackendFunction
         push(@function, "            responseMessage->setObject(\"data\", responseData);");
         push(@function, "        }");
     }
-    push(@function, "        m_inspectorController->inspectorClient()->sendMessageToFrontend(responseMessage->toJSONString());");
+    push(@function, "        m_inspectorAgent->inspectorClient()->sendMessageToFrontend(responseMessage->toJSONString());");
     push(@function, "    }");
 
 
@@ -447,7 +448,7 @@ void ${backendClassName}::reportProtocolError(const long callId, const String& e
     RefPtr<InspectorArray> errors = InspectorArray::create();
     errors->pushString(errorText);
     message->setArray("errors", errors);
-    m_inspectorController->inspectorClient()->sendMessageToFrontend(message->toJSONString());
+    m_inspectorAgent->inspectorClient()->sendMessageToFrontend(message->toJSONString());
 }
 EOF
     return split("\n", $reportProtocolError);
diff --git a/Source/WebCore/inspector/InspectorController.h b/Source/WebCore/inspector/InspectorController.h
index ab8377d..4a0847f 100644
--- a/Source/WebCore/inspector/InspectorController.h
+++ b/Source/WebCore/inspector/InspectorController.h
@@ -145,8 +145,32 @@ public:
     void reuseFrontend();
     void disconnectFrontend();
 
+    InspectorController* inspectorAgent() { return this; }
     InspectorConsoleAgent* consoleAgent() const { return m_consoleAgent.get(); }
+    InspectorCSSAgent* cssAgent() const { return m_cssAgent.get(); }
     InspectorDOMAgent* domAgent() const { return m_domAgent.get(); }
+    InjectedScriptHost* injectedScriptAgent() const { return m_injectedScriptHost.get(); }
+    InspectorResourceAgent* resourceAgent() const { return m_resourceAgent.get(); }
+    InspectorRuntimeAgent* runtimeAgent() const { return m_runtimeAgent.get(); }
+    InspectorTimelineAgent* timelineAgent() { return m_timelineAgent.get(); }
+#if ENABLE(DATABASE)
+    InspectorDatabaseAgent* databaseAgent() const { return m_databaseAgent.get(); }
+#endif
+#if ENABLE(DOM_STORAGE)
+    InspectorDOMStorageAgent* domStorageAgent() const { return m_domStorageAgent.get(); }
+#endif
+#if ENABLE(FILE_SYSTEM)
+    InspectorFileSystemAgent* fileSystemAgent() const { return m_fileSystemAgent.get(); }
+#endif
+#if ENABLE(JAVASCRIPT_DEBUGGER)
+    InspectorBrowserDebuggerAgent* browserDebuggerAgent() const { return m_browserDebuggerAgent.get(); }
+    InspectorDebuggerAgent* debuggerAgent() const { return m_debuggerAgent.get(); }
+    InspectorProfilerAgent* profilerAgent() const { return m_profilerAgent.get(); }
+#endif
+#if ENABLE(OFFLINE_WEB_APPLICATIONS)
+    InspectorApplicationCacheAgent* applicationCacheAgent() { return m_applicationCacheAgent.get(); }
+#endif
+
 
     bool searchingForNodeInPage() const;
     void mouseDidMoveOverElement(const HitTestResult&, unsigned modifierFlags);
@@ -163,15 +187,10 @@ public:
 
     void startTimelineProfiler();
     void stopTimelineProfiler();
-    InspectorTimelineAgent* timelineAgent() { return m_timelineAgent.get(); }
 
     void getCookies(RefPtr<InspectorArray>* cookies, WTF::String* cookiesString);
     void deleteCookie(const String& cookieName, const String& domain);
 
-#if ENABLE(OFFLINE_WEB_APPLICATIONS)
-    InspectorApplicationCacheAgent* applicationCacheAgent() { return m_applicationCacheAgent.get(); }
-#endif
-
     void mainResourceFiredLoadEvent(DocumentLoader*, const KURL&);
     void mainResourceFiredDOMContentEvent(DocumentLoader*, const KURL&);
 
@@ -336,7 +355,6 @@ private:
     bool m_attachDebuggerWhenShown;
     OwnPtr<InspectorDebuggerAgent> m_debuggerAgent;
     OwnPtr<InspectorBrowserDebuggerAgent> m_browserDebuggerAgent;
-
     OwnPtr<InspectorProfilerAgent> m_profilerAgent;
 #endif
     OwnPtr<HTTPHeaderMap> m_extraHeaders;

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list