MacViews: UnhandledKeyboardEventHandler stub

MacViews: UnhandledKeyboardEventHandler stub

Fix for missing UnhandledKeyboardEventHandler::HandleNativeKeyboardEvent on
Mac, in preparation for compiling chrome/browser/ui/views for MacViews.
Consolidate duplicated linux/windows code to be shared code and stub out
the Mac implementation.

BUG=425229
Committed: https://crrev.com/f5d10685b975dd11be315133a099a8a980ad0716
Cr-Commit-Position: refs/heads/master@{#301198}

[Andre, 2014-10-24 16:50:12]

andresantoso@chromium.org changed reviewers:
+ sky@chromium.org

[Andre, 2014-10-24 16:50:12]

sky@ PTAL.
We are starting work to build chrome/browser/ui/views for MacViews.

[sky, 2014-10-24 18:11:19]

https://codereview.chromium.org/679743002/diff/1/ui/views/controls/webview/un...
File ui/views/controls/webview/unhandled_keyboard_event_handler.cc (right):

https://codereview.chromium.org/679743002/diff/1/ui/views/controls/webview/un...
ui/views/controls/webview/unhandled_keyboard_event_handler.cc:57: if
(event.os_event && !event.skip_in_browser)
The win side previously didn't have the !event.skip_in_browser.

[Andre, 2014-10-24 18:36:02]

https://codereview.chromium.org/679743002/diff/1/ui/views/controls/webview/un...
File ui/views/controls/webview/unhandled_keyboard_event_handler.cc (right):

https://codereview.chromium.org/679743002/diff/1/ui/views/controls/webview/un...
ui/views/controls/webview/unhandled_keyboard_event_handler.cc:57: if
(event.os_event && !event.skip_in_browser)
On 2014/10/24 18:11:19, sky wrote:
> The win side previously didn't have the !event.skip_in_browser.

Right, sorry I should have added a comment about that.
skip_in_browser is only set in browser_test_utils.cc, ime_adapter_android.cc,
and render_widget_host_view_mac.mm, so it is currently never set for
windows other than testing.
And it seemed to me like the flag should be checked regardless of platform.

[sky, 2014-10-24 19:31:04]

Ok, LGTM

[Andre, 2014-10-24 20:15:02]

The CQ bit was checked by andresantoso@chromium.org

[commit-bot: I haz the power, 2014-10-24 20:16:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/679743002/1

[commit-bot: I haz the power, 2014-10-24 21:36:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-24 21:37:33]

Patchset 1 (id:??) landed as
https://crrev.com/f5d10685b975dd11be315133a099a8a980ad0716
Cr-Commit-Position: refs/heads/master@{#301198}