Title: [182734] trunk/Source/WebCore
- Revision
- 182734
- Author
- [email protected]
- Date
- 2015-04-13 08:40:42 -0700 (Mon, 13 Apr 2015)
Log 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.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (182733 => 182734)
--- trunk/Source/WebCore/ChangeLog 2015-04-13 15:06:23 UTC (rev 182733)
+++ trunk/Source/WebCore/ChangeLog 2015-04-13 15:40:42 UTC (rev 182734)
@@ -1,3 +1,14 @@
+2015-04-13 Darin Adler <[email protected]>
+
+ 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 <[email protected]>
[cmake] Add ENABLE(ATTACHMENT_ELEMENT) to the build system
Modified: trunk/Source/WebCore/loader/NavigationScheduler.cpp (182733 => 182734)
--- trunk/Source/WebCore/loader/NavigationScheduler.cpp 2015-04-13 15:06:23 UTC (rev 182733)
+++ trunk/Source/WebCore/loader/NavigationScheduler.cpp 2015-04-13 15:40:42 UTC (rev 182734)
@@ -386,8 +386,7 @@
// 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;
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
