[webkit-reviews] review requested: [Bug 14729] [gtk] Implement FrameLoaderClientGdk::createFrame : [Attachment 15754] Implement ScrollView similiar to win/qt

[bugzilla-daemon at webkit.org]

Holger Freyther <freyther at handhelds.org> has asked  for review:
Bug 14729: [gtk] Implement FrameLoaderClientGdk::createFrame
http://bugs.webkit.org/show_bug.cgi?id=14729

Attachment 15754: Implement ScrollView similiar to win/qt
http://bugs.webkit.org/attachment.cgi?id=15754&action=edit

------- Additional Comments from Holger Freyther <freyther at handhelds.org>
Compared to the previous patch:

-Make it compile on case-sensitive filesystems (PlatformScrollbar.h vs.
PlatformScrollBar.h)
-Add (again) refcounting to PlatformScrollbar and ScrollView
-Implement forall and unrealize in WebKitGtkPage to make issues on exit go
away.
-Fixup some comments.

The biggest part of the patch is copy and paste from the windows port. The
controversal change is in Widget to use one of the two possible IntRect
conversion methods. I think this change is fine, other approaches would be to
move this code to PlatformScrollbar or to subclass PlatformScrollbar and use
that subclss in ScrollView.

I think this is commit-worthy.