[webkit-reviews] review requested: [Bug 39217] Add an optional "starting node' parameter to scrollRecursively and scrollOverflow of EventHandler : [Attachment 56250] patch v2.1

[bugzilla-daemon at webkit.org]

Antonio Gomes (tonikitoo) <tonikitoo at webkit.org> has asked  for review:
Bug 39217: Add an optional "starting node' parameter to scrollRecursively and
scrollOverflow of EventHandler
https://bugs.webkit.org/show_bug.cgi?id=39217

Attachment 56250: patch v2.1
https://bugs.webkit.org/attachment.cgi?id=56250&action=review

------- Additional Comments from Antonio Gomes (tonikitoo)
<tonikitoo at webkit.org>
(In reply to comment #4)
> (From update of attachment 56242 [details])
> I don't think the name "fromNode" is clear enough. Argument names should be
noun phrases; there's no such thing as a "from node". Maybe "starting node" is
the right name?

Same as patch v2, but changed "fromNode" to 'startingNode"