Title: [144601] trunk/Source/WebCore
Revision
144601
Author
apav...@chromium.org
Date
2013-03-03 23:44:50 -0800 (Sun, 03 Mar 2013)

Log Message

Web Inspector: [PageAgent] can't find frame by security origin.
https://bugs.webkit.org/show_bug.cgi?id=110849

Use toRawString() in order to compare Frames' SecurityOrigins for storage-related goals.

Reviewed by Vsevolod Vlasov.

* inspector/InspectorPageAgent.cpp:
(WebCore::InspectorPageAgent::findFrameWithSecurityOrigin):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (144600 => 144601)


--- trunk/Source/WebCore/ChangeLog	2013-03-04 07:28:53 UTC (rev 144600)
+++ trunk/Source/WebCore/ChangeLog	2013-03-04 07:44:50 UTC (rev 144601)
@@ -1,5 +1,17 @@
 2013-03-03  Alexander Pavlov  <apav...@chromium.org>
 
+        Web Inspector: [PageAgent] can't find frame by security origin.
+        https://bugs.webkit.org/show_bug.cgi?id=110849
+
+        Use toRawString() in order to compare Frames' SecurityOrigins for storage-related goals.
+
+        Reviewed by Vsevolod Vlasov.
+
+        * inspector/InspectorPageAgent.cpp:
+        (WebCore::InspectorPageAgent::findFrameWithSecurityOrigin):
+
+2013-03-03  Alexander Pavlov  <apav...@chromium.org>
+
         Unreviewed, rolling out r144455.
         http://trac.webkit.org/changeset/144455
         https://bugs.webkit.org/show_bug.cgi?id=111165

Modified: trunk/Source/WebCore/inspector/InspectorPageAgent.cpp (144600 => 144601)


--- trunk/Source/WebCore/inspector/InspectorPageAgent.cpp	2013-03-04 07:28:53 UTC (rev 144600)
+++ trunk/Source/WebCore/inspector/InspectorPageAgent.cpp	2013-03-04 07:44:50 UTC (rev 144601)
@@ -933,16 +933,10 @@
 
 Frame* InspectorPageAgent::findFrameWithSecurityOrigin(const String& originRawString)
 {
-    RefPtr<SecurityOrigin> securityOriginPtr = SecurityOrigin::createFromString(originRawString);
-    SecurityOrigin* securityOrigin = securityOriginPtr.get();
-    bool isLocal = securityOrigin->isLocal();
     for (Frame* frame = m_page->mainFrame(); frame; frame = frame->tree()->traverseNext()) {
         RefPtr<SecurityOrigin> documentOrigin = frame->document()->securityOrigin();
-        // Emulate the !enforceFilePathSeparation for security origins.
-        if (isLocal && documentOrigin->isLocal())
+        if (documentOrigin->toRawString() == originRawString)
             return frame;
-        if (documentOrigin->equal(securityOrigin))
-            return frame;
     }
     return 0;
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to