[webkit-qt] API review, comments needed!
tonikitoo (Antonio Gomes)
tonikitoo at gmail.com
Fri Mar 5 08:16:43 PST 2010
On Fri, Mar 5, 2010 at 11:16 AM, Simon Hausmann
<simon.hausmann at nokia.com> wrote:
> Hi,
>
> we just went through the diff of the header files between Qt 4.6's WebKit and
> the trunk. It's a relatively small diff, but nevertheless taking another look
> at it revealed a few interesting issues. Here are the comments and questions
> we collected. Please comment :)
>
>
> QWebFrame::scrollRecursively:
>
> * Implicit dependency on last mouse move event is bad. The position should
> become an explicit function parameter.
>
> * The use of "recursion" in the name is confusing, it suggests recursive
> behaviour to child frames. However the behaviour is to continue scrolling the
> parent frames.
>
> * The documentation does not explain what the return value is.
>
> * The auto test only checks successful scrolling. It never tests the case
> of the function
> returning false.
I agree and even I asked for that comment to be added in the bug in
https://bugs.webkit.org/show_bug.cgi?id=32668#c13
More information about the webkit-qt
mailing list