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

Unified Diff: extensions/renderer/resources/guest_view/guest_view_container.js

Issue 972313002: Make <webview> use out-of-process iframe architecture. (Closed) Base URL: ssh://saopaulo.wat/mnt/dev/shared/src@testoopif2z-better-chrome
Patch Set: Revert unused changes from previous attempt + more cleanup. Created 5 years, 10 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/guest_view/guest_view_container.js
diff --git a/extensions/renderer/resources/guest_view/guest_view_container.js b/extensions/renderer/resources/guest_view/guest_view_container.js
index 0107a7e9bc7881a0a8c11ff2ebbbcaabc58be943..091f58b564e572489a3915aa4d5add5ab8a75c34 100644
--- a/extensions/renderer/resources/guest_view/guest_view_container.js
+++ b/extensions/renderer/resources/guest_view/guest_view_container.js
@@ -64,6 +64,10 @@ GuestViewContainer.prototype.createBrowserPluginElement = function() {
return browserPluginElement;
};
+GuestViewContainer.prototype.getBrowserPluginElement = function() {
+ return privates(this).browserPluginElement;
+};
+
GuestViewContainer.prototype.setupFocusPropagation = function() {
if (!this.element.hasAttribute('tabIndex')) {
// GuestViewContainer needs a tabIndex in order to be focusable.
@@ -134,11 +138,12 @@ GuestViewContainer.prototype.onElementResize = function(oldWidth, oldHeight,
// Registers the browser plugin <object> custom element. |viewType| is the
// name of the specific guestview container (e.g. 'webview').
function registerBrowserPluginElement(viewType) {
- var proto = Object.create(HTMLObjectElement.prototype);
+ //var proto = Object.create(HTMLObjectElement.prototype);
+ var proto = Object.create(HTMLIFrameElement.prototype);
proto.createdCallback = function() {
- this.setAttribute('type', 'application/browser-plugin');
- this.setAttribute('id', 'browser-plugin-' + IdGenerator.GetNextId());
+ //this.setAttribute('type', 'application/browser-plugin');
+ //this.setAttribute('id', 'browser-plugin-' + IdGenerator.GetNextId());
this.style.width = '100%';
this.style.height = '100%';
};
@@ -148,6 +153,7 @@ function registerBrowserPluginElement(viewType) {
var unused = this.nonExistentAttribute;
};
+ /*
proto.attributeChangedCallback = function(name, oldValue, newValue) {
var internal = privates(this).internal;
if (!internal) {
@@ -155,10 +161,12 @@ function registerBrowserPluginElement(viewType) {
}
internal.handleBrowserPluginAttributeMutation(name, oldValue, newValue);
};
+ */
GuestViewContainer[viewType + 'BrowserPlugin'] =
DocumentNatives.RegisterElement(viewType + 'browserplugin',
- {extends: 'object', prototype: proto});
+ //{extends: 'object', prototype: proto});
+ {extends: 'iframe', prototype: proto});
delete proto.createdCallback;
delete proto.attachedCallback;

Powered by Google App Engine
This is Rietveld 408576698