[Webkit-unassigned] [Bug 131878] New: Fix AVPlayerController leak on pages with video

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=131878

           Summary: Fix AVPlayerController leak on pages with video
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: joepeck at webkit.org
                CC: jer.noble at apple.com, jeremyj-wk at apple.com


I was seeing leaks of an AVPlayerController allocated with this stack:

        ...
        | WebKit::WebContext::createWebPage(WebKit::PageClient&, WebKit::WebPageConfiguration)
        | WebKit::WebProcessProxy::createWebPage(WebKit::PageClient&, WebKit::WebPageConfiguration const&)
        | WebKit::WebPageProxy::create(WebKit::PageClient&, WebKit::WebProcessProxy&, unsigned long long, WebKit::WebPageConfiguration const&)
        | WebKit::WebPageProxy::WebPageProxy(WebKit::PageClient&, WebKit::WebProcessProxy&, unsigned long long, WebKit::WebPageConfiguration const&)
        | WebKit::WebVideoFullscreenManagerProxy::create(WebKit::WebPageProxy&)
        | WebKit::WebVideoFullscreenManagerProxy::WebVideoFullscreenManagerProxy(WebKit::WebPageProxy&)
        | WebCore::WebVideoFullscreenInterfaceAVKit::setWebVideoFullscreenModel(WebCore::WebVideoFullscreenModel*)
        | -[WebAVPlayerController init]
        | _objc_rootAlloc
        | class_createInstance
        | calloc
        | malloc_zone_calloc 

The allocation was:

    self.playerControllerProxy = [[classAVPlayerController alloc] init];

This is a (retain) property which will +1 the incoming object, so the alloc/init needs to be balanced.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.