Handle closing web contents in ExtensionOptionsGuest

chrome/browser/resources/extensions/extension_options_overlay.js

Index: chrome/browser/resources/extensions/extension_options_overlay.js
diff --git a/chrome/browser/resources/extensions/extension_options_overlay.js b/chrome/browser/resources/extensions/extension_options_overlay.js
index 899098d625ec446767be32d832602b97367fd582..29597ac0c3b45e8c5a99fa684cd444b4963bde0a 100644
--- a/chrome/browser/resources/extensions/extension_options_overlay.js
+++ b/chrome/browser/resources/extensions/extension_options_overlay.js
@@ -64,6 +64,10 @@ cr.define('extensions', function() {
       extensionoptions.extension = extensionId;
       extensionoptions.autosize = 'on';
 
+      extensionoptions.onclose = function() {
+        this.handleDismiss_();

[Fady Samuel, 2014/08/20 14:53:55]

The intent here is to remove the extensionoptions element when window.close is
called from the guest?

[ericzeng, 2014/08/20 17:26:00]

The higher level intent is to close the entire overlay when window.close is
called from the guest. Imagine if an extension options page in a tab had a
'save' button that saves the settings and then calls window.close() to close
itself. I want the same behavior for the overlay - when the extension options
page wants to close itself, its <extensionoptions> is removed and the overlay
closed. 

+      }.bind(this);
+
       // TODO(ericzeng): Resize in a non-jarring way.
       extensionoptions.onsizechanged = function(evt) {
         $('extension-options-overlay').style.width = evt.width;