[Webkit-unassigned] [Bug 31448] Menu button shows wrong content in Chromium

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Nov 16 10:28:58 PST 2009


https://bugs.webkit.org/show_bug.cgi?id=31448


Darin Fisher (:fishd, Google) <fishd at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #43116|review?                     |review-
               Flag|                            |




--- Comment #4 from Darin Fisher (:fishd, Google) <fishd at chromium.org>  2009-11-16 10:28:57 PST ---
(From update of attachment 43116)
> Index: WebKit/chromium/src/WebViewImpl.cpp

> +    } else if (focusedNode) {
> +        location = focusedNode->getRect().bottomLeft();
> +    } else {

nit: no brackets around single line statements.

otherwise, LGTM

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list