Make contentsToScreen and screenToContents cross-platform
Created attachment 23987 [details] Patch
Created attachment 23988 [details] Patch #2
Comment on attachment 23988 [details] Patch #2 +++ WebKit/gtk/WebCoreSupport/ChromeClientGtk.h (working copy) @@ -88,6 +88,8 @@ namespace WebKit { virtual void updateBackingStore(); virtual void repaint(const WebCore::IntRect&, bool contentChanged, bool immediate = false); + virtual IntPoint screenToWindow(const IntPoint& p) const; + virtual IntRect windowToScreen(const IntRect& r) const; Looks like you're missing some "WebCore::" on the types. The parameter names aren't needed here, either. +++ WebKit/qt/WebCoreSupport/ChromeClientQt.h (working copy) @@ -104,7 +104,9 @@ namespace WebCore { virtual void updateBackingStore(); virtual void repaint(const IntRect&, bool contentChanged, bool immediate = false); - + virtual IntPoint screenToWindow(const IntPoint& p) const; + virtual IntRect windowToScreen(const IntRect& r) const; Parameter names can be omitted. +++ WebKit/wx/WebKitSupport/ChromeClientWx.h (working copy) @@ -99,6 +99,8 @@ public: virtual void updateBackingStore(); virtual void repaint(const IntRect&, bool contentChanged, bool immediate = false); + virtual IntPoint screenToWindow(const IntPoint& p) const; + virtual IntRect windowToScreen(const IntRect& r) const; Ditto. r=me
Fixed in r37153.