On Thu, Sep 3, 2009 at 1:58 PM, <fin...@chromium.org> wrote:
>
> Author: fin...@chromium.org
> Date: Thu Sep 3 13:58:01 2009
> New Revision: 25367
>
> Log:
> Fix 9867: Activating the previous/next buttons with the keyboard in the find
> bar should not change the focus.
>
> Add param const Event& event to ButtonPressed, so that recipients can find
> out more about the event that generated the ButtonPress message.
>
>
> Modified: trunk/src/views/controls/button/native_button.cc
> ==============================================================================
> --- trunk/src/views/controls/button/native_button.cc (original)
> +++ trunk/src/views/controls/button/native_button.cc Thu Sep 3 13:58:01
> 2009
> @@ -4,8 +4,14 @@
>
> #include "views/controls/button/native_button.h"
>
> +#if defined(OS_WIN)
> +#include <atlbase.h>
> +#include <atlapp.h> // for GET_X/Y_LPARAM
> +#endif
> +
Please don't add more dependencies to ATL. We're in the process of
removing said dependencies. GET_X/Y_LPARAM can be replaced by
MAKEPOINTS which doesn't require ATL.
--
James Hawkins
--~--~---------~--~----~------------~-------~--~----~
Chromium Developers mailing list: chromium-dev@googlegroups.com
View archives, change email options, or unsubscribe:
http://groups.google.com/group/chromium-dev
-~----------~----~----~----~------~----~------~--~---
