JavaScriptBench
diff --git a/Collapse file
‎Source/WebKit/ChangeLog‎
Copy file name to clipboardExpand all lines: Source/WebKit/ChangeLog
+14Lines changed: 14 additions & 0 deletions b/Collapse file
‎Source/WebKit/ChangeLog‎
Copy file name to clipboardExpand all lines: Source/WebKit/ChangeLog
+14Lines changed: 14 additions & 0 deletions
diff --git a/Collapse file
‎Source/WebKit/UIProcess/mac/DisplayLink.cpp‎
Copy file name to clipboardExpand all lines: Source/WebKit/UIProcess/mac/DisplayLink.cpp
-1Lines changed: 0 additions & 1 deletion b/Collapse file
‎Source/WebKit/UIProcess/mac/DisplayLink.cpp‎
Copy file name to clipboardExpand all lines: Source/WebKit/UIProcess/mac/DisplayLink.cpp
-1Lines changed: 0 additions & 1 deletion
diff --git a/Collapse file
‎Source/WebKit/UIProcess/mac/DisplayLink.h‎
Copy file name to clipboardExpand all lines: Source/WebKit/UIProcess/mac/DisplayLink.h
-1Lines changed: 0 additions & 1 deletion b/Collapse file
‎Source/WebKit/UIProcess/mac/DisplayLink.h‎
Copy file name to clipboardExpand all lines: Source/WebKit/UIProcess/mac/DisplayLink.h
-1Lines changed: 0 additions & 1 deletion
@@ -1,3 +1,17 @@
+2018-04-17  Conrad Shultz  <conrad_shultz@apple.com>
+
+        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  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         [Extra zoom mode] Double tap to zoom should account for text legibility in extra zoom mode
 
@@ -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);
 
@@ -49,7 +49,6 @@ class DisplayLink {
     static CVReturn displayLinkCallback(CVDisplayLinkRef, const CVTimeStamp*, const CVTimeStamp*, CVOptionFlags, CVOptionFlags*, void* data);
 
     CVDisplayLinkRef m_displayLink { nullptr };
-    Ref<WebPageProxy> m_webPageProxy;
     HashSet<unsigned> m_observers;
 };
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,6 @@`
`36`	`36`	`namespace WebKit {`
`37`	`37`
`38`	`38`	`DisplayLink::DisplayLink(WebCore::PlatformDisplayID displayID, WebPageProxy& webPageProxy)`
`39`		`- : m_webPageProxy(webPageProxy)`
`40`	`39`	`{`
`41`	`40`	`RELEASE_ASSERT(hasProcessPrivilege(ProcessPrivilege::CanCommunicateWithWindowServer));`
`42`	`41`	`CVReturn error = CVDisplayLinkCreateWithCGDisplay(displayID, &m_displayLink);`