[webkit-reviews] review denied: [Bug 58503] Make more functions static local in EventHandlers.cpp : [Attachment 89598] cleanup

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 26 15:40:57 PDT 2011


Nate Chapin <japhet at chromium.org> has denied Ryosuke Niwa <rniwa at webkit.org>'s
request for review:
Bug 58503: Make more functions static local in EventHandlers.cpp
https://bugs.webkit.org/show_bug.cgi?id=58503

Attachment 89598: cleanup
https://bugs.webkit.org/attachment.cgi?id=89598&action=review

------- Additional Comments from Nate Chapin <japhet at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=89598&action=review

r- based mostly on the previous comments

> Source/WebCore/page/EventHandler.cpp:2519
> +    DEFINE_STATIC_LOCAL(AtomicString, Down, ("Down"));
> +    DEFINE_STATIC_LOCAL(AtomicString, Up, ("Up"));
> +    DEFINE_STATIC_LOCAL(AtomicString, Left, ("Left"));
> +    DEFINE_STATIC_LOCAL(AtomicString, Right, ("Right"));

I know you're just moving this code, but these variable names should be
lowercase.

Also, is it worth putting these in a map?


More information about the webkit-reviews mailing list