Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(110)

Unified Diff: extensions/renderer/resources/web_view.js

Issue 564913003: Moving web_view.js to extensions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/renderer/resources/web_view.js
diff --git a/chrome/renderer/resources/extensions/web_view.js b/extensions/renderer/resources/web_view.js
similarity index 98%
rename from chrome/renderer/resources/extensions/web_view.js
rename to extensions/renderer/resources/web_view.js
index 63872ce214fd25e787c1093aaac06a257e2cbcdd..610c65f1b1a7f1752a9b736717a6424654ef3201 100644
--- a/chrome/renderer/resources/extensions/web_view.js
+++ b/extensions/renderer/resources/web_view.js
@@ -10,7 +10,6 @@ var DocumentNatives = requireNative('document_natives');
var GuestViewInternal =
require('binding').Binding.create('guestViewInternal').generate();
var IdGenerator = requireNative('id_generator');
-var ChromeWebView = require('chromeWebViewInternal').ChromeWebView
// TODO(lazyboy): Rename this to WebViewInternal and call WebViewInternal
// something else.
var WebView = require('webViewInternal').WebView;
@@ -1012,6 +1011,12 @@ window.addEventListener('readystatechange', function listener(event) {
}, useCapture);
/**
+ * Implemented when the ChromeWebView API is available.
+ * @private
+ */
+WebViewInternal.prototype.maybeGetChromeWebViewEvents = function() {};
+
+/**
* Implemented when the experimental API is available.
* @private
*/
@@ -1026,24 +1031,11 @@ WebViewInternal.prototype.maybeGetExperimentalPermissions = function() {
};
/**
- * Calls to show contextmenu right away instead of dispatching a 'contextmenu'
- * event.
- * This will be overridden in web_view_experimental.js to implement contextmenu
- * API.
- */
-WebViewInternal.prototype.maybeHandleContextMenu = function(e, webViewEvent) {
- var requestId = e.requestId;
- // Setting |params| = undefined will show the context menu unmodified, hence
- // the 'contextmenu' API is disabled for stable channel.
- var params = undefined;
- ChromeWebView.showContextMenu(this.guestInstanceId, requestId, params);
-};
-
-/**
* Implemented when the experimental API is available.
* @private
*/
-WebViewInternal.prototype.setupExperimentalContextMenus = function() {};
+WebViewInternal.prototype.setupExperimentalContextMenus = function() {
+};
exports.WebView = WebView;
exports.WebViewInternal = WebViewInternal;
« no previous file with comments | « extensions/renderer/resources/extensions_renderer_resources.grd ('k') | extensions/renderer/resources/web_view_deny.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698