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

Unified Diff: chrome/renderer/resources/extensions/web_view_events.js

Issue 440463002: Fix display:none issue for <webview>. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync Created 6 years, 4 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: chrome/renderer/resources/extensions/web_view_events.js
diff --git a/chrome/renderer/resources/extensions/web_view_events.js b/chrome/renderer/resources/extensions/web_view_events.js
index 546dff294f74349acfd2f42fe8e593585350976c..0331038698faf2f30f91421ffdb1b8694f6c82f9 100644
--- a/chrome/renderer/resources/extensions/web_view_events.js
+++ b/chrome/renderer/resources/extensions/web_view_events.js
@@ -20,6 +20,7 @@ var CreateEvent = function(name) {
};
var FrameNameChangedEvent = CreateEvent('webViewInternal.onFrameNameChanged');
+var PluginDestroyedEvent = CreateEvent('webViewInternal.onPluginDestroyed');
var WebRequestMessageEvent = CreateEvent('webViewInternal.onMessage');
// WEB_VIEW_EVENTS is a map of stable <webview> DOM event names to their
@@ -192,6 +193,7 @@ function WebViewEvents(webViewInternal, viewInstanceId) {
// Sets up events.
WebViewEvents.prototype.setup = function() {
this.setupFrameNameChangedEvent();
+ this.setupPluginDestroyedEvent();
this.setupWebRequestEvents();
this.webViewInternal.setupExperimentalContextMenus();
@@ -208,6 +210,13 @@ WebViewEvents.prototype.setupFrameNameChangedEvent = function() {
}, {instanceId: self.viewInstanceId});
};
+WebViewEvents.prototype.setupPluginDestroyedEvent = function() {
+ var self = this;
+ PluginDestroyedEvent.addListener(function(e) {
+ self.webViewInternal.onPluginDestroyed();
+ }, {instanceId: self.viewInstanceId});
+};
Fady Samuel 2014/08/25 18:55:34 I'm trying to switch away from self to bind(this).
lazyboy 2014/08/25 19:16:44 Done.
+
WebViewEvents.prototype.setupWebRequestEvents = function() {
var self = this;
var request = {};

Powered by Google App Engine
This is Rietveld 408576698