Fixed the <webview> context menu bug.

chrome/renderer/resources/extensions/chrome_web_view_experimental.js

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This module implements experimental API for <webview>.
 // See web_view.js for details.
 //
 // <webview> Chrome Experimental API is only available on canary and dev
 // channels of Chrome.
 
 var ChromeWebView = require('chromeWebViewInternal').ChromeWebView;
 var ChromeWebViewSchema =
     requireNative('schema_registry').GetSchema('chromeWebViewInternal');
 var ContextMenusSchema =
     requireNative('schema_registry').GetSchema('contextMenus');
 var CreateEvent = require('guestViewEvents').CreateEvent;
 var EventBindings = require('event_bindings');
 var idGeneratorNatives = requireNative('id_generator');
 var MessagingNatives = requireNative('messaging_natives');
 var utils = require('utils');
+var WebViewEvents = require('webViewEvents').WebViewEvents;
 var WebViewImpl = require('webView').WebViewImpl;
 
 function GetUniqueSubEventName(eventName) {
   return eventName + '/' + idGeneratorNatives.GetNextId();
 }
 
 // This is the only "webViewInternal.onClicked" named event for this renderer.
 //
 // Since we need an event per <webview>, we define events with suffix
 // (subEventName) in each of the <webview>. Behind the scenes, this event is
@@ skipping 56 matching lines @@
 WebViewContextMenusImpl.prototype.update = function() {
   var args = $Array.concat([this.viewInstanceId_], $Array.slice(arguments));
   return $Function.apply(ChromeWebView.contextMenusUpdate, null, args);
 };
 
 var WebViewContextMenus = utils.expose(
     'WebViewContextMenus', WebViewContextMenusImpl,
     { functions: ['create', 'remove', 'removeAll', 'update'] });
 
 /** @private */
-WebViewImpl.prototype.maybeHandleContextMenu = function(e, webViewEvent) {
-  var requestId = e.requestId;
+WebViewEvents.prototype.handleContextMenu = function(event, eventName) {
+  var webViewEvent = this.makeDomEvent(event, eventName);
+  var requestId = event.requestId;
   // Construct the event.menu object.
   var actionTaken = false;
   var validateCall = function() {
     var ERROR_MSG_CONTEXT_MENU_ACTION_ALREADY_TAKEN = '<webview>: ' +
         'An action has already been taken for this "contextmenu" event.';
 
     if (actionTaken) {
       throw new Error(ERROR_MSG_CONTEXT_MENU_ACTION_ALREADY_TAKEN);
     }
     actionTaken = true;
   };
   var menu = {
     show: function(items) {
       validateCall();
       // TODO(lazyboy): WebViewShowContextFunction doesn't do anything useful
       // with |items|, implement.
-      ChromeWebView.showContextMenu(this.guest.getId(), requestId, items);
+      ChromeWebView.showContextMenu(this.view.guest.getId(), requestId, items);
     }.bind(this)
   };
   webViewEvent.menu = menu;
-  var element = this.element;
+  var element = this.view.element;
   var defaultPrevented = !element.dispatchEvent(webViewEvent);
   if (actionTaken) {
     return;
   }
   if (!defaultPrevented) {
     actionTaken = true;
     // The default action is equivalent to just showing the context menu as is.
-    ChromeWebView.showContextMenu(this.guest.getId(), requestId, undefined);
+    ChromeWebView.showContextMenu(
+        this.view.guest.getId(), requestId, undefined);
 
     // TODO(lazyboy): Figure out a way to show warning message only when
     // listeners are registered for this event.
   } //  else we will ignore showing the context menu completely.
 };
 
 /** @private */
 WebViewImpl.prototype.setupExperimentalContextMenus = function() {
   var createContextMenus = function() {
     return function() {
@@ skipping 30 matching lines @@
 
   // Expose <webview>.contextMenus object.
   Object.defineProperty(
       this.element,
       'contextMenus',
       {
         get: createContextMenus(),
         enumerable: true
       });
 };