[Webkit-unassigned] [Bug 164437] Support TouchBar in WebKit

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Nov 7 17:36:47 PST 2016


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

--- Comment #27 from Beth Dakin <bdakin at apple.com> ---
(In reply to comment #26)
> Comment on attachment 294046 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=294046&action=review
> 
> > Source/WebKit/mac/WebView/WebView.mm:9577
> > +- (void)webViewAdditionsWillDestroyView
> 
> Could we rename this to -webViewWillDestroyView? (It looks like the
> "Additions" terminology is being dropped from other places).
> Alternately, maybe we don't need a separate function on the WebView for
> performing cleanup in -[WebView dealloc] -- I think it makes sense just to
> move the cleanup logic directly to -dealloc and remove the
> -webViewWillDestroyView helper.
> 
> This also applies to the WebKit2 case (webViewImplAdditionsWillDestroyView)

Good find. I removed the functions entirely.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20161108/5d3fed14/attachment.html>


More information about the webkit-unassigned mailing list