Store WeakPtr<Frame> instead of Frame*
Created attachment 413244 [details] Patch
Created attachment 413247 [details] Patch
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Comment on attachment 413247 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=413247&action=review > Source/WebCore/bindings/js/WindowProxy.h:53 > + WEBCORE_EXPORT AbstractFrame* frame() const; Why not keeping it inline?
Comment on attachment 413247 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=413247&action=review >> Source/WebCore/bindings/js/WindowProxy.h:53 >> + WEBCORE_EXPORT AbstractFrame* frame() const; > > Why not keeping it inline? That would require including AbstractFrame.h to build successfully.
Tools/Scripts/svn-apply failed to apply attachment 413247 [details] to trunk. Please resolve the conflicts and upload a new patch.
http://trac.webkit.org/r269435
<rdar://problem/71079886>
It looks like the changes in https://trac.webkit.org/changeset/269435/webkit broke Mac builds builds: https://build.webkit.org/builders/Apple-BigSur-Release-Build/builds/2839 ./loader/FrameLoader.cpp:389:12: error: no viable conversion from returned value of type 'WeakPtr<WebCore::Frame>' to function return type 'WebCore::Frame *' ./loader/FrameLoader.cpp:394:12: error: no viable conversion from returned value of type 'const WeakPtr<WebCore::Frame>' to function return type 'const WebCore::Frame *'
https://trac.webkit.org/changeset/269439/webkit