RESOLVED FIXED 21282
Make contentsToScreen and screenToContents cross-platform 
https://bugs.webkit.org/show_bug.cgi?id=21282
Summary Make contentsToScreen and screenToContents cross-platform
Dave Hyatt
Reported 2008-10-01 13:00:29 PDT
Make contentsToScreen and screenToContents cross-platform
Attachments
Patch (20.24 KB, patch)
2008-10-01 13:03 PDT, Dave Hyatt 		no flags
DetailsFormatted DiffDiff
Patch #2 (20.28 KB, patch)
2008-10-01 13:08 PDT, Dave Hyatt 		aroben: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dave Hyatt
Comment 1 2008-10-01 13:03:42 PDT
Created attachment 23987 [details] Patch
Dave Hyatt
Comment 2 2008-10-01 13:08:17 PDT
Created attachment 23988 [details] Patch #2
Adam Roben (:aroben)
Comment 3 2008-10-01 13:26:49 PDT
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
Dave Hyatt
Comment 4 2008-10-01 13:34:56 PDT
Fixed in r37153.
Note You need to log in before you can comment on or make changes to this bug.