Title: [182784] releases/WebKitGTK/webkit-2.8/Source/WebCore
- Revision
- 182784
- Author
- carlo...@webkit.org
- Date
- 2015-04-14 01:27:45 -0700 (Tue, 14 Apr 2015)
Log Message
Merge r182734 - 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: releases/WebKitGTK/webkit-2.8/Source/WebCore/ChangeLog (182783 => 182784)
--- releases/WebKitGTK/webkit-2.8/Source/WebCore/ChangeLog 2015-04-14 08:20:40 UTC (rev 182783)
+++ releases/WebKitGTK/webkit-2.8/Source/WebCore/ChangeLog 2015-04-14 08:27:45 UTC (rev 182784)
@@ -1,3 +1,14 @@
+2015-04-13 Darin Adler <da...@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-12 Simon Fraser <simon.fra...@apple.com>
Selects don't scroll at some aspect ratios
Modified: releases/WebKitGTK/webkit-2.8/Source/WebCore/loader/NavigationScheduler.cpp (182783 => 182784)
--- releases/WebKitGTK/webkit-2.8/Source/WebCore/loader/NavigationScheduler.cpp 2015-04-14 08:20:40 UTC (rev 182783)
+++ releases/WebKitGTK/webkit-2.8/Source/WebCore/loader/NavigationScheduler.cpp 2015-04-14 08:27:45 UTC (rev 182784)
@@ -366,8 +366,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
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
