Factor out Lookup invocation
Created attachment 260022 [details] Patch
Created attachment 260076 [details] Patch
Created attachment 260077 [details] Patch
Created attachment 260079 [details] Patch
Comment on attachment 260079 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=260079&action=review > Source/WebKit2/UIProcess/mac/WKImmediateActionController.mm:470 > + return (id<NSImmediateActionAnimationController>)DictionaryLookup::animationControllerForPopup(dictionaryPopupInfo, _wkView, [self](TextIndicator& textIndicator) { Why do you need the cast here?
(In reply to comment #5) > Comment on attachment 260079 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=260079&action=review > > > Source/WebKit2/UIProcess/mac/WKImmediateActionController.mm:470 > > + return (id<NSImmediateActionAnimationController>)DictionaryLookup::animationControllerForPopup(dictionaryPopupInfo, _wkView, [self](TextIndicator& textIndicator) { > > Why do you need the cast here? I don't, anymore! (In a previous version of the patch, it always returned void*)
http://trac.webkit.org/changeset/189052