Bug 40185

Summary: WebPageProxy should keep a weak reference to the PageClient
Product: WebKit Reporter: Ada Chan <adachan>
Component: WebKit2Assignee: Adam Roben (:aroben) <aroben>
Severity: Normal CC: andersca, aroben, kenneth, sam, zoltan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Description Flags
Patch andersca: review+

Description Ada Chan 2010-06-04 13:21:09 PDT
Right now WebPageProxy stores an OwnPtr to the PageClient.  That is not appropriate because WebPageProxy does not create the PageClient and it can be set to something that it should not own.  For example, on Windows, it's currently being set to the WebView itself.  This will cause a crash because WebView has a reference to WebPageProxy which has an OwnPtr to its PageClient, which is the WebView.
Comment 1 Adam Roben (:aroben) 2010-07-27 13:33:30 PDT
Seems like WKView should own the PageClient instead.
Comment 2 Adam Roben (:aroben) 2010-07-27 14:01:46 PDT
Created attachment 62743 [details]
Comment 3 Adam Roben (:aroben) 2010-07-27 14:08:58 PDT
Committed r64155: <http://trac.webkit.org/changeset/64155>