[webkit-reviews] review granted: [Bug 103389] PDFPlugin: "Show Definition" context menu item should be implemented : [Attachment 176614] patch (much simplified)

[bugzilla-daemon at webkit.org]

mitz at webkit.org <mitz at webkit.org> has granted Tim Horton
<timothy_horton at apple.com>'s request for review:
Bug 103389: PDFPlugin: "Show Definition" context menu item should be
implemented
https://bugs.webkit.org/show_bug.cgi?id=103389

Attachment 176614: patch (much simplified)
https://bugs.webkit.org/attachment.cgi?id=176614&action=review

------- Additional Comments from mitz at webkit.org <mitz at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=176614&action=review


> Source/WebKit2/WebProcess/Plugins/PDF/PDFPlugin.mm:780
> +    dictionaryPopupInfo.origin =
convertFromPDFViewToRootView(IntPoint(point));

Is the IntPoint() necessary?