Changeset 44420 in webkit

Timestamp:

Jun 4, 2009 11:34:23 AM (15 years ago)

Author:

bfulgham@webkit.org

Message:

2009-06-04 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=26180
Add a fast path for SecurityOrigin::equal. If "other == this" (where
other is the other security origin), then we really don't need to do
all the other (expensive) comparisons. We know it's equal.

• page/SecurityOrigin.cpp: (WebCore::SecurityOrigin::equal):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/SecurityOrigin.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-06-04  Jeremy Orlow  <jorlow@chromium.org>
+
+        Reviewed by Darin Adler.
+
+        https://bugs.webkit.org/show_bug.cgi?id=26180
+        Add a fast path for SecurityOrigin::equal. If "other == this" (where
+        other is the other security origin), then we really don't need to do
+        all the other (expensive) comparisons. We know it's equal.
+
+        * page/SecurityOrigin.cpp:
+        (WebCore::SecurityOrigin::equal):
+
 2009-06-03  David Hyatt  <hyatt@apple.com>
 

trunk/WebCore/page/SecurityOrigin.cpp

@@ -278 +278 @@
 bool SecurityOrigin::equal(const SecurityOrigin* other) const 
 {
+    if (other == this)
+        return true;
+    
     if (!isSameSchemeHostPort(other))
         return false;