Title: [228385] branches/safari-605-branch/Source/WebCore
- Revision
- 228385
- Author
- jmarc...@apple.com
- Date
- 2018-02-12 11:51:53 -0800 (Mon, 12 Feb 2018)
Log Message
Cherry-pick r228045. rdar://problem/37465611
Modified Paths
Diff
Modified: branches/safari-605-branch/Source/WebCore/ChangeLog (228384 => 228385)
--- branches/safari-605-branch/Source/WebCore/ChangeLog 2018-02-12 18:58:16 UTC (rev 228384)
+++ branches/safari-605-branch/Source/WebCore/ChangeLog 2018-02-12 19:51:53 UTC (rev 228385)
@@ -1,5 +1,19 @@
2018-02-12 Jason Marcell <jmarc...@apple.com>
+ Cherry-pick r228045. rdar://problem/37465611
+
+ 2018-02-02 Daniel Bates <daba...@apple.com>
+
+ Clean up FrameLoader::receivedFirstData()
+ https://bugs.webkit.org/show_bug.cgi?id=182361
+
+ Reviewed by Andy Estes.
+
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::receivedFirstData):
+
+2018-02-12 Jason Marcell <jmarc...@apple.com>
+
Cherry-pick r228326. rdar://problem/37460483
2018-02-09 Youenn Fablet <you...@apple.com>
Modified: branches/safari-605-branch/Source/WebCore/loader/FrameLoader.cpp (228384 => 228385)
--- branches/safari-605-branch/Source/WebCore/loader/FrameLoader.cpp 2018-02-12 18:58:16 UTC (rev 228384)
+++ branches/safari-605-branch/Source/WebCore/loader/FrameLoader.cpp 2018-02-12 19:51:53 UTC (rev 228385)
@@ -667,24 +667,22 @@
dispatchDidClearWindowObjectsInAllWorlds();
dispatchGlobalObjectAvailableInAllWorlds();
- if (m_documentLoader) {
- auto& title = m_documentLoader->title();
- if (!title.string.isNull())
- m_client.dispatchDidReceiveTitle(title);
- }
-
if (!m_documentLoader)
return;
+ auto& documentLoader = *m_documentLoader;
+ auto& title = documentLoader.title();
+ if (!title.string.isNull())
+ m_client.dispatchDidReceiveTitle(title);
+
ASSERT(m_frame.document());
auto& document = *m_frame.document();
- ASSERT(m_frame.document());
- LinkLoader::loadLinksFromHeader(m_documentLoader->response().httpHeaderField(HTTPHeaderName::Link), m_frame.document()->url(), *m_frame.document(), LinkLoader::MediaAttributeCheck::MediaAttributeEmpty);
+ LinkLoader::loadLinksFromHeader(documentLoader.response().httpHeaderField(HTTPHeaderName::Link), document.url(), document, LinkLoader::MediaAttributeCheck::MediaAttributeEmpty);
double delay;
String urlString;
- if (!parseHTTPRefresh(m_documentLoader->response().httpHeaderField(HTTPHeaderName::Refresh), delay, urlString))
+ if (!parseHTTPRefresh(documentLoader.response().httpHeaderField(HTTPHeaderName::Refresh), delay, urlString))
return;
auto completedURL = urlString.isEmpty() ? document.url() : document.completeURL(urlString);
if (!protocolIsJavaScript(completedURL))
@@ -691,7 +689,7 @@
m_frame.navigationScheduler().scheduleRedirect(document, delay, completedURL);
else {
auto message = "Refused to refresh " + document.url().stringCenterEllipsizedToLength() + " to a _javascript_: URL";
- m_frame.document()->addConsoleMessage(MessageSource::Security, MessageLevel::Error, message);
+ document.addConsoleMessage(MessageSource::Security, MessageLevel::Error, message);
}
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes