Title: [230770] tags/Safari-606.1.14/Source/WebKit
- Revision
- 230770
- Author
- [email protected]
- Date
- 2018-04-18 11:42:37 -0700 (Wed, 18 Apr 2018)
Log Message
Cherry-pick r230747. rdar://problem/39468022
WebKit::DisplayLink maintains a strong reference to WebPageProxy, creating a reference cycle
https://bugs.webkit.org/show_bug.cgi?id=184718
Reviewed by Tim Horton.
It turns out that the m_webPageProxy back-reference in DisplayLink, which was creating a
reference cycle, wasn't ever read, so we can just remove it.
* UIProcess/mac/DisplayLink.cpp:
(WebKit::DisplayLink::DisplayLink):
* UIProcess/mac/DisplayLink.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@230747 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: tags/Safari-606.1.14/Source/WebKit/ChangeLog (230769 => 230770)
--- tags/Safari-606.1.14/Source/WebKit/ChangeLog 2018-04-18 18:02:03 UTC (rev 230769)
+++ tags/Safari-606.1.14/Source/WebKit/ChangeLog 2018-04-18 18:42:37 UTC (rev 230770)
@@ -1,3 +1,36 @@
+2018-04-18 Kocsen Chung <[email protected]>
+
+ Cherry-pick r230747. rdar://problem/39468022
+
+ WebKit::DisplayLink maintains a strong reference to WebPageProxy, creating a reference cycle
+ https://bugs.webkit.org/show_bug.cgi?id=184718
+
+ Reviewed by Tim Horton.
+
+ It turns out that the m_webPageProxy back-reference in DisplayLink, which was creating a
+ reference cycle, wasn't ever read, so we can just remove it.
+
+ * UIProcess/mac/DisplayLink.cpp:
+ (WebKit::DisplayLink::DisplayLink):
+ * UIProcess/mac/DisplayLink.h:
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@230747 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2018-04-17 Conrad Shultz <[email protected]>
+
+ WebKit::DisplayLink maintains a strong reference to WebPageProxy, creating a reference cycle
+ https://bugs.webkit.org/show_bug.cgi?id=184718
+
+ Reviewed by Tim Horton.
+
+ It turns out that the m_webPageProxy back-reference in DisplayLink, which was creating a
+ reference cycle, wasn't ever read, so we can just remove it.
+
+ * UIProcess/mac/DisplayLink.cpp:
+ (WebKit::DisplayLink::DisplayLink):
+ * UIProcess/mac/DisplayLink.h:
+
2018-04-17 Adrian Perez de Castro <[email protected]>
[GTK][WPE] Build failure due to presence of Avahi's <dns_sd.h> header
Modified: tags/Safari-606.1.14/Source/WebKit/UIProcess/mac/DisplayLink.cpp (230769 => 230770)
--- tags/Safari-606.1.14/Source/WebKit/UIProcess/mac/DisplayLink.cpp 2018-04-18 18:02:03 UTC (rev 230769)
+++ tags/Safari-606.1.14/Source/WebKit/UIProcess/mac/DisplayLink.cpp 2018-04-18 18:42:37 UTC (rev 230770)
@@ -36,7 +36,6 @@
namespace WebKit {
DisplayLink::DisplayLink(WebCore::PlatformDisplayID displayID, WebPageProxy& webPageProxy)
- : m_webPageProxy(webPageProxy)
{
RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));
CVReturn error = CVDisplayLinkCreateWithCGDisplay(displayID, &m_displayLink);
Modified: tags/Safari-606.1.14/Source/WebKit/UIProcess/mac/DisplayLink.h (230769 => 230770)
--- tags/Safari-606.1.14/Source/WebKit/UIProcess/mac/DisplayLink.h 2018-04-18 18:02:03 UTC (rev 230769)
+++ tags/Safari-606.1.14/Source/WebKit/UIProcess/mac/DisplayLink.h 2018-04-18 18:42:37 UTC (rev 230770)
@@ -49,7 +49,6 @@
static CVReturn displayLinkCallback(CVDisplayLinkRef, const CVTimeStamp*, const CVTimeStamp*, CVOptionFlags, CVOptionFlags*, void* data);
CVDisplayLinkRef m_displayLink { nullptr };
- Ref<WebPageProxy> m_webPageProxy;
HashSet<unsigned> m_observers;
};
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
