[Mac] Factor out keyboard shortcut handling from ChromeEventProcessingWindow.

ui/views/controls/webview/webview.cc

Index: ui/views/controls/webview/webview.cc
diff --git a/ui/views/controls/webview/webview.cc b/ui/views/controls/webview/webview.cc
index c9e3105cd60f6e066da5703303b044c5562b0733..55703b6889e230f113b79a353915b4e63a14c5ac 100644
--- a/ui/views/controls/webview/webview.cc
+++ b/ui/views/controls/webview/webview.cc
@@ -18,6 +18,7 @@
 #include "ui/views/controls/native/native_view_host.h"
 #include "ui/views/focus/focus_manager.h"
 #include "ui/views/views_delegate.h"
+#include "ui/views/widget/native_widget_private.h"
 
 namespace views {
 
@@ -249,6 +250,15 @@ void WebView::RenderProcessHostDestroyed(content::RenderProcessHost* host) {
 ////////////////////////////////////////////////////////////////////////////////
 // WebView, content::WebContentsDelegate implementation:
 
+void WebView::HandleKeyboardEvent(

[tapted, 2015/08/18 07:22:13]

is this override only to support views_examples? (if so, we might want to
separate that out to a different CL - I think in practice  it's more common for
webviews to be used like 

WebContents* web_contents = foo();
webview->SetWebContents(web_contents);

and in this case the WebView isn't the delegate for the WebContents, so this
won't get called.

An alternative fix for ViewsExamples might be to make WebViewExample the
WebContentsDelegate, and make use of UnhandledKeyboardEventHandler (i.e. same as
NativeAppWindowViews and friends). Of course we'd need to implement
UnhandledKeyboardEventHandler::HandleNativeKeyboardEvent like you were before
(and we may still need RepostNativeEvent to be implmeneted as well, but it looks
like WebDialog is the only caller and AFAIK it's not used much outside of
ChromeOS.... if at all even: InlineLoginDialog looks like dead code outside
ChromeOS).

[jackhou1, 2015/08/25 06:31:12]

Done.

Left it in this CL because it's a fairly small change, let me know if you prefer
to have it split out.

+    content::WebContents* source,
+    const content::NativeWebKeyboardEvent& event) {
+  if (!event.os_event)
+    return;
+
+  GetWidget()->native_widget_private()->RepostNativeEvent(event.os_event);
+}
+
 bool WebView::EmbedsFullscreenWidget() const {
   DCHECK(wc_owner_.get());
   return embed_fullscreen_widget_mode_enabled_;