[webkit-reviews] review requested: [Bug 16809] Clicking a scrollbar blurs the currently focused element : [Attachment 54576] patch v2

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 28 10:25:41 PDT 2010


Antonio Gomes (tonikitoo) <tonikitoo at webkit.org> has asked  for review:
Bug 16809: Clicking a scrollbar blurs the currently focused element
https://bugs.webkit.org/show_bug.cgi?id=16809

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

------- Additional Comments from Antonio Gomes (tonikitoo)
<tonikitoo at webkit.org>
Based on my arguments on Darin's concern, I kept the same logic as in patch
v1.1, but:

> Need braces around the body of this if statement.

Fixed.

> Since in that case we are trying to affect only the focus
> behavior then I think the code should go inside the mousedownEvent if
> statement.

Fixed.


More information about the webkit-reviews mailing list