From 183d87c0c20d5bea0fb40378c4437a811c63f7da Mon Sep 17 00:00:00 2001
From: dbeam <dbeam@chromium.org>
Date: Tue, 11 Aug 2015 11:50:23 -0700
Subject: [PATCH] Remove focus event listeners from un-focusable node

This UI used to live in a world where its <ul> element had a tabindex=0
or something.  No longer.

R=hcarmona@chromium.org
BUG=518053

Review URL: https://codereview.chromium.org/1273033004

Cr-Commit-Position: refs/heads/master@{#342855}
---
 .../resources/extensions/extension_error.js        | 23 +---------------------
 1 file changed, 1 insertion(+), 22 deletions(-)

diff --git a/chrome/browser/resources/extensions/extension_error.js b/chrome/browser/resources/extensions/extension_error.js
index ab66f77d3fbd..e0cf7ab7f27e 100644
--- a/chrome/browser/resources/extensions/extension_error.js
+++ b/chrome/browser/resources/extensions/extension_error.js
@@ -176,9 +176,7 @@ cr.define('extensions', function() {
 
       this.focusGrid_ = new cr.ui.FocusGrid();
       this.listContents_ = this.querySelector('.extension-error-list-contents');
-      this.listContents_.addEventListener('focus', this.onFocus_.bind(this));
-      this.listContents_.addEventListener('focusin',
-                                          this.onFocusin_.bind(this));
+
       errors.forEach(this.addError_, this);
 
       this.addEventListener('highlightExtensionError', function(e) {
@@ -356,25 +354,6 @@ cr.define('extensions', function() {
           new CustomEvent('activeExtensionErrorChanged',
                           {bubbles: true, detail: node ? node.error : null}));
     },
-
-    /**
-     * The grid should not be focusable once it or an element inside it is
-     * focused. This is necessary to allow tabbing out of the grid in reverse.
-     * @private
-     */
-    onFocusin_: function() {
-      this.listContents_.tabIndex = -1;
-    },
-
-    /**
-     * Focus the first focusable row when tabbing into the grid for the
-     * first time.
-     * @private
-     */
-    onFocus_: function() {
-      var activeRow = this.listContents_.querySelector('.focus-row-active');
-      activeRow.getEquivalentElement(null).focus();
-    },
   };
 
   return {
-- 
2.11.4.GIT