extensions: reinstate trash focus/closing logic after switch to

chrome/browser/resources/extensions/extension_list.js

Index: chrome/browser/resources/extensions/extension_list.js
diff --git a/chrome/browser/resources/extensions/extension_list.js b/chrome/browser/resources/extensions/extension_list.js
index a9363e1ff0137a2e907509c31a06d38e2aa6e31b..5beb8d7065b40d09f341c0af7bad2db2e4ed44b7 100644
--- a/chrome/browser/resources/extensions/extension_list.js
+++ b/chrome/browser/resources/extensions/extension_list.js
@@ -466,6 +466,10 @@ cr.define('extensions', function() {
                                     function() {
           // TODO(devlin): What should we do if the uninstall fails?
           this.uninstallIsShowing_ = false;
+          if (trash.classList.contains('mouse-clicked'))
+            trash.blur();
+          if (chrome.extension.lastError)

[Devlin, 2015/03/03 00:17:40]

Maybe a comment like:
// chrome.extension.lastError is set if the uninstall failed or was canceled;
// in either case, the entry in the list is still present.
?

[Dan Beam, 2015/03/03 00:25:43]

Added comment to that effect.

+            trash.classList.remove('open');
         }.bind(this));
       }.bind(this));
       row.querySelector('.enable-controls').appendChild(trash);
@@ -899,13 +903,6 @@ cr.define('extensions', function() {
     },
   };
 
-  ExtensionList.uninstallCancel = function() {
-    var trash = document.querySelector('.trash.open');
-    if (trash.classList.contains('mouse-clicked'))
-      trash.blur();
-    trash.classList.remove('open');
-  };
-
   return {
     ExtensionList: ExtensionList
   };