Fix some lingering WebUI focus issues.

chrome/browser/resources/extensions/extension_commands_overlay.js

Index: chrome/browser/resources/extensions/extension_commands_overlay.js
diff --git a/chrome/browser/resources/extensions/extension_commands_overlay.js b/chrome/browser/resources/extensions/extension_commands_overlay.js
index ab56eb4e1719d0ef3a5c43d1ebc754ff27abc07d..190330b6b25056e4252557dc32f9fca6985965ee 100644
--- a/chrome/browser/resources/extensions/extension_commands_overlay.js
+++ b/chrome/browser/resources/extensions/extension_commands_overlay.js
@@ -34,8 +34,9 @@ cr.define('extensions', function() {
       this.extensionCommandList_ = new ExtensionCommandList(
           /**@type {HTMLDivElement} */($('extension-command-list')));
 
-      $('extension-commands-dismiss').addEventListener('click',
-          this.handleDismiss_.bind(this));

[Dan Beam, 2015/06/04 03:30:29]

this was short-circuiting the focus restoration code.

+      $('extension-commands-dismiss').addEventListener('click', function() {
+        cr.dispatchEvent(overlay, 'cancelOverlay');

[dmazzoni, 2015/06/04 03:34:56]

Just curious, is there a reason this is dispatchEvent and the others are
dispatchSimpleEvent?

[Dan Beam, 2015/06/04 18:50:35]

Done. (and closure compiled for first time and actually tried this code)

+      });
 
       // The ExtensionList will update us with its data, so we don't need to
       // handle that here.