Bug 926477 - Replace className usages with classList in browser-places.js. r=jaws
authorRene Amrhein <waldion@hotmail.com>
Thu, 12 Dec 2013 12:37:51 -0500
changeset 160194 764e877917a4eb5dca0aefafa480a5700a25de15
parent 160193 d9107edc306a93f28c23b89b4145a3b8403ee752
child 160195 4bc1aa2876211d44f47983494f314dbf6ed5fbfd
push id25826
push userkwierso@gmail.com
push dateFri, 13 Dec 2013 02:14:34 +0000
treeherdermozilla-central@f551c8244289 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjaws
bugs926477
milestone29.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 926477 - Replace className usages with classList in browser-places.js. r=jaws
browser/base/content/browser-places.js
--- a/browser/base/content/browser-places.js
+++ b/browser/base/content/browser-places.js
@@ -97,19 +97,19 @@ var StarUI = {
           break;
         }
         switch (aEvent.keyCode) {
           case KeyEvent.DOM_VK_ESCAPE:
             if (!this._element("editBookmarkPanelContent").hidden)
               this.cancelButtonOnCommand();
             break;
           case KeyEvent.DOM_VK_RETURN:
-            if (aEvent.target.className == "expander-up" ||
-                aEvent.target.className == "expander-down" ||
-                aEvent.target.id == "editBMPanel_newFolderButton") {
+            if (aEvent.target.classList.contains("expander-up") ||
+                aEvent.target.classList.contains("expander-down") ||
+                aEvent.target.id == "editBMPanel_newFolderButton")  {
               //XXX Why is this necessary? The defaultPrevented check should
               //    be enough.
               break;
             }
             this.panel.hidePopup();
             break;
         }
         break;