Restrict extensionview to chrome-extension://

extensions/renderer/resources/guest_view/extension_view.js

 // Copyright 2015 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 the ExtensionView <extensionview>.
 
 var GuestViewContainer = require('guestViewContainer').GuestViewContainer;
 var ExtensionViewConstants =
     require('extensionViewConstants').ExtensionViewConstants;
+var ExtensionViewEvents = require('extensionViewEvents').ExtensionViewEvents;
 var ExtensionViewInternal =
     require('extensionViewInternal').ExtensionViewInternal;
 
 function ExtensionViewImpl(extensionviewElement) {
   GuestViewContainer.call(this, extensionviewElement, 'extensionview');
   this.setupExtensionViewAttributes();
+
+  new ExtensionViewEvents(this, this.viewInstanceId);
 }
 
 ExtensionViewImpl.prototype.__proto__ = GuestViewContainer.prototype;
 
 ExtensionViewImpl.VIEW_TYPE = 'ExtensionView';
 
 ExtensionViewImpl.setupElement = function(proto) {
-  var apiMethods = [
-    'navigate'
-  ];
+  var apiMethods = ExtensionViewImpl.getApiMethods();
 
   GuestViewContainer.forwardApiMethods(proto, apiMethods);
 };
 
 ExtensionViewImpl.prototype.createGuest = function() {
   this.guest.create(this.buildParams(), function() {
     this.attachWindow();
   }.bind(this));
 };
 
-ExtensionViewImpl.prototype.onElementAttached = function() {
-  this.attributes[ExtensionViewConstants.ATTRIBUTE_SRC].parse();
-};
-
 ExtensionViewImpl.prototype.buildContainerParams = function() {
   var params = {};
   for (var i in this.attributes) {
     params[i] = this.attributes[i].getValue();
   }
   return params;
 };
 
 // This observer monitors mutations to attributes of the <extensionview>.
 ExtensionViewImpl.prototype.handleAttributeMutation = function(
     attributeName, oldValue, newValue) {
   if (!this.attributes[attributeName])
     return;
 
-  // Let the changed attribute handle its own mutation;
+  // Let the changed attribute handle its own mutation.
   this.attributes[attributeName].maybeHandleMutation(oldValue, newValue);
 };
 
 ExtensionViewImpl.prototype.onElementDetached = function() {
   this.guest.destroy();
-  this.attributes[ExtensionViewConstants.ATTRIBUTE_SRC].reset();
+
+  // Reset all attributes.
+  for (var i in this.attributes) {
+    this.attributes[i].reset();
+  }
+};
+
+// Updates src upon loadcommit.
+ExtensionViewImpl.prototype.onLoadCommit = function(url) {
+  var oldValue =
+      this.attributes[ExtensionViewConstants.ATTRIBUTE_SRC].getValue();
+  var newValue = url;
+  if (oldValue != newValue) {

[Fady Samuel, 2015/02/17 23:20:36]

No need for this. If this is in webview then it's cruft. I'd just call:

this.attributes[ExtensionViewConstants.ATTRIBUTE_SRC].
    setValueIgnoreMutation(newValue);

[apacible, 2015/02/18 00:04:11]

Done.

+    this.attributes[ExtensionViewConstants.ATTRIBUTE_SRC].
+        setValueIgnoreMutation(newValue);
+  }
 };
 
 GuestViewContainer.registerElement(ExtensionViewImpl);
 
 // Exports.
 exports.ExtensionViewImpl = ExtensionViewImpl;