Title: [142869] trunk/Source/WebCore
Revision
142869
Author
apav...@chromium.org
Date
2013-02-14 04:47:16 -0800 (Thu, 14 Feb 2013)

Log Message

Web Inspector: Consistently use SecurityOrigin::toRawString() for serialization across the backend code
https://bugs.webkit.org/show_bug.cgi?id=109801

Reviewed by Yury Semikhatsky.

No new tests, as existing tests cover the change.

* inspector/InspectorAgent.cpp:
(WebCore::InspectorAgent::didClearWindowObjectInWorld):
* inspector/InspectorIndexedDBAgent.cpp:
(WebCore::InspectorIndexedDBAgent::requestDatabaseNamesForFrame):
* inspector/InspectorPageAgent.cpp:
(WebCore::InspectorPageAgent::buildObjectForFrame):
* inspector/PageRuntimeAgent.cpp:
(WebCore::PageRuntimeAgent::notifyContextCreated):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (142868 => 142869)


--- trunk/Source/WebCore/ChangeLog	2013-02-14 12:35:09 UTC (rev 142868)
+++ trunk/Source/WebCore/ChangeLog	2013-02-14 12:47:16 UTC (rev 142869)
@@ -1,3 +1,21 @@
+2013-02-14  Alexander Pavlov  <apav...@chromium.org>
+
+        Web Inspector: Consistently use SecurityOrigin::toRawString() for serialization across the backend code
+        https://bugs.webkit.org/show_bug.cgi?id=109801
+
+        Reviewed by Yury Semikhatsky.
+
+        No new tests, as existing tests cover the change.
+
+        * inspector/InspectorAgent.cpp:
+        (WebCore::InspectorAgent::didClearWindowObjectInWorld):
+        * inspector/InspectorIndexedDBAgent.cpp:
+        (WebCore::InspectorIndexedDBAgent::requestDatabaseNamesForFrame):
+        * inspector/InspectorPageAgent.cpp:
+        (WebCore::InspectorPageAgent::buildObjectForFrame):
+        * inspector/PageRuntimeAgent.cpp:
+        (WebCore::PageRuntimeAgent::notifyContextCreated):
+
 2013-02-14  Sergio Villar Senin  <svil...@igalia.com>
 
         Add logging support to IndexedDB for non-Chromium platforms

Modified: trunk/Source/WebCore/inspector/InspectorAgent.cpp (142868 => 142869)


--- trunk/Source/WebCore/inspector/InspectorAgent.cpp	2013-02-14 12:35:09 UTC (rev 142868)
+++ trunk/Source/WebCore/inspector/InspectorAgent.cpp	2013-02-14 12:47:16 UTC (rev 142869)
@@ -86,7 +86,7 @@
     if (m_injectedScriptForOrigin.isEmpty())
         return;
 
-    String origin = frame->document()->securityOrigin()->toString();
+    String origin = frame->document()->securityOrigin()->toRawString();
     String script = m_injectedScriptForOrigin.get(origin);
     if (!script.isEmpty())
         m_injectedScriptManager->injectScript(script, mainWorldScriptState(frame));

Modified: trunk/Source/WebCore/inspector/InspectorIndexedDBAgent.cpp (142868 => 142869)


--- trunk/Source/WebCore/inspector/InspectorIndexedDBAgent.cpp	2013-02-14 12:35:09 UTC (rev 142868)
+++ trunk/Source/WebCore/inspector/InspectorIndexedDBAgent.cpp	2013-02-14 12:47:16 UTC (rev 142869)
@@ -658,7 +658,7 @@
         requestCallback->sendFailure("Could not obtain database names.");
         return;
     }
-    idbRequest->addEventListener(eventNames().successEvent, GetDatabaseNamesCallback::create(requestCallback, document->securityOrigin()->toString()), false);
+    idbRequest->addEventListener(eventNames().successEvent, GetDatabaseNamesCallback::create(requestCallback, document->securityOrigin()->toRawString()), false);
 }
 
 void InspectorIndexedDBAgent::requestDatabase(ErrorString* errorString, const String& frameId, const String& databaseName, PassRefPtr<RequestDatabaseCallback> requestCallback)

Modified: trunk/Source/WebCore/inspector/InspectorPageAgent.cpp (142868 => 142869)


--- trunk/Source/WebCore/inspector/InspectorPageAgent.cpp	2013-02-14 12:35:09 UTC (rev 142868)
+++ trunk/Source/WebCore/inspector/InspectorPageAgent.cpp	2013-02-14 12:47:16 UTC (rev 142869)
@@ -1057,7 +1057,7 @@
         frameObject->setName(name);
     }
     // FIXME: Make this field non-optional. https://bugs.webkit.org/show_bug.cgi?id=80857
-    frameObject->setSecurityOrigin(frame->document()->securityOrigin()->toString());
+    frameObject->setSecurityOrigin(frame->document()->securityOrigin()->toRawString());
 
     return frameObject;
 }

Modified: trunk/Source/WebCore/inspector/PageRuntimeAgent.cpp (142868 => 142869)


--- trunk/Source/WebCore/inspector/PageRuntimeAgent.cpp	2013-02-14 12:35:09 UTC (rev 142868)
+++ trunk/Source/WebCore/inspector/PageRuntimeAgent.cpp	2013-02-14 12:47:16 UTC (rev 142869)
@@ -179,7 +179,7 @@
 {
     ASSERT(securityOrigin || isPageContext);
     int executionContextId = injectedScriptManager()->injectedScriptIdFor(scriptState);
-    String name = securityOrigin ? securityOrigin->toString() : "";
+    String name = securityOrigin ? securityOrigin->toRawString() : "";
     m_frontend->executionContextCreated(ExecutionContextDescription::create()
         .setId(executionContextId)
         .setIsPageContext(isPageContext)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to