Changeset 147671 in webkit

Timestamp:

Apr 4, 2013 2:57:58 PM (11 years ago)

Author:

ap@apple.com

Message:

REGRESSION (r146580): Assertion using SecurityOrigin::create, URL string is not valid
​https://bugs.webkit.org/show_bug.cgi?id=113961

Reviewed by Geoff Garen.

• page/SecurityOrigin.cpp: (WebCore::SecurityOrigin::createFromDatabaseIdentifier): (WebCore::SecurityOrigin::create): Don't use ParsedURLString, because ​http://host:80/ is normalized to ​http://host/, so it's not a parsed URL string.

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-04-04  Alexey Proskuryakov  <ap@apple.com>
+
+        REGRESSION (r146580): Assertion using SecurityOrigin::create, URL string is not valid
+        https://bugs.webkit.org/show_bug.cgi?id=113961
+
+        Reviewed by Geoff Garen.
+
+        * page/SecurityOrigin.cpp:
+        (WebCore::SecurityOrigin::createFromDatabaseIdentifier):
+        (WebCore::SecurityOrigin::create):
+        Don't use ParsedURLString, because http://host:80/ is normalized to http://host/,
+        so it's not a parsed URL string.
+
 2013-04-04  Sheriff Bot  <webkit.review.bot@gmail.com>
 

trunk/Source/WebCore/page/SecurityOrigin.cpp

@@ -530 +530 @@
     
     host = decodeURLEscapeSequences(host);
-    return create(KURL(ParsedURLString, protocol + "://" + host + ":" + String::number(port) + "/"));
+    return create(KURL(KURL(), protocol + "://" + host + ":" + String::number(port) + "/"));
 }
 
@@ -538 +538 @@
         createUnique();
     String decodedHost = decodeURLEscapeSequences(host);
-    return create(KURL(ParsedURLString, protocol + "://" + host + ":" + String::number(port) + "/"));
+    return create(KURL(KURL(), protocol + "://" + host + ":" + String::number(port) + "/"));
 }