Title: [181575] trunk/Source/WebCore
- Revision
- 181575
- Author
- beid...@apple.com
- Date
- 2015-03-16 13:05:14 -0700 (Mon, 16 Mar 2015)
Log Message
Addressing additional review feedback after http://trac.webkit.org/changeset/181565
https://bugs.webkit.org/show_bug.cgi?id=142604
Reviewed by Darin Adler.
* loader/icon/IconController.cpp:
(WebCore::IconController::startLoader): Null check page()
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (181574 => 181575)
--- trunk/Source/WebCore/ChangeLog 2015-03-16 19:40:10 UTC (rev 181574)
+++ trunk/Source/WebCore/ChangeLog 2015-03-16 20:05:14 UTC (rev 181575)
@@ -1,3 +1,13 @@
+2015-03-16 Brady Eidson <beid...@apple.com>
+
+ Addressing additional review feedback after http://trac.webkit.org/changeset/181565
+ https://bugs.webkit.org/show_bug.cgi?id=142604
+
+ Reviewed by Darin Adler.
+
+ * loader/icon/IconController.cpp:
+ (WebCore::IconController::startLoader): Null check page()
+
2015-03-16 Roger Fong <roger_f...@apple.com>
[Win] Unreviewed build fix attempt after r181571.
Modified: trunk/Source/WebCore/loader/icon/IconController.cpp (181574 => 181575)
--- trunk/Source/WebCore/loader/icon/IconController.cpp 2015-03-16 19:40:10 UTC (rev 181574)
+++ trunk/Source/WebCore/loader/icon/IconController.cpp 2015-03-16 20:05:14 UTC (rev 181575)
@@ -161,7 +161,7 @@
if (iconDatabase().supportsAsynchronousMode()) {
// FIXME (<rdar://problem/9168605>) - We should support in-memory-only private browsing icons in asynchronous icon database mode.
- if (m_frame.page()->usesEphemeralSession())
+ if (m_frame.page() && m_frame.page()->usesEphemeralSession())
return;
m_frame.loader().documentLoader()->getIconLoadDecisionForIconURL(urlString);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes