Changeset 182734 in webkit

Timestamp:

Apr 13, 2015, 8:40:42 AM (10 years ago)

Author:

Darin Adler

Message:

Remove needless recreation of URL in NavigationScheduler::scheduleLocationChange
​https://bugs.webkit.org/show_bug.cgi?id=143662

Reviewed by Sam Weinig.

• loader/NavigationScheduler.cpp:

(WebCore::NavigationScheduler::scheduleLocationChange): Removed unnecessary code
to convert a URL to a String and then back into a URL.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/NavigationScheduler.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2015-04-13  Darin Adler  <darin@apple.com>
+
+        Remove needless recreation of URL in NavigationScheduler::scheduleLocationChange
+        https://bugs.webkit.org/show_bug.cgi?id=143662
+
+        Reviewed by Sam Weinig.
+
+        * loader/NavigationScheduler.cpp:
+        (WebCore::NavigationScheduler::scheduleLocationChange): Removed unnecessary code
+        to convert a URL to a String and then back into a URL.
+
 2015-04-13  Csaba Osztrogonác  <ossy@webkit.org>
 

trunk/Source/WebCore/loader/NavigationScheduler.cpp

@@ -387 +387 @@
     // If the URL we're going to navigate to is the same as the current one, except for the
     // fragment part, we don't need to schedule the location change.
-    URL parsedURL(ParsedURLString, url);
-    if (parsedURL.hasFragmentIdentifier() && equalIgnoringFragmentIdentifier(m_frame.document()->url(), parsedURL)) {
+    if (url.hasFragmentIdentifier() && equalIgnoringFragmentIdentifier(m_frame.document()->url(), url)) {
         loader.changeLocation(securityOrigin, m_frame.document()->completeURL(url), referrer, lockHistory, lockBackForwardList, false, AllowNavigationToInvalidURL::No);
         return;