Bug 952042 - Remove fake focus-ring after findbar is closed. r=mikedeboer, a=lsblakk
authorTom Schuster <evilpies@gmail.com>
Fri, 24 Jan 2014 13:08:38 +0100
changeset 176165 12c9107dd05095bc32469f846452241a43654b67
parent 176164 b78c42b80c28e1b016416f6641aacadea214f232
child 176166 14d893faf45b523638b426f9bae76e4d16931a14
push id445
push userffxbld
push dateMon, 10 Mar 2014 22:05:19 +0000
treeherdermozilla-release@dc38b741b04e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmikedeboer, lsblakk
bugs952042
milestone28.0a2
Bug 952042 - Remove fake focus-ring after findbar is closed. r=mikedeboer, a=lsblakk
toolkit/modules/Finder.jsm
--- a/toolkit/modules/Finder.jsm
+++ b/toolkit/modules/Finder.jsm
@@ -102,25 +102,22 @@ Finder.prototype = {
       let result = found ? Ci.nsITypeAheadFind.FIND_FOUND
                          : Ci.nsITypeAheadFind.FIND_NOTFOUND;
       this._notify(aWord, result, false, false);
     }
   },
 
   enableSelection: function() {
     this._fastFind.setSelectionModeAndRepaint(Ci.nsISelectionController.SELECTION_ON);
+    this._restoreOriginalOutline();
   },
 
   removeSelection: function() {
-    let fastFind = this._fastFind;
-
-    fastFind.collapseSelection();
+    this._fastFind.collapseSelection();
     this.enableSelection();
-
-    this._restoreOriginalOutline();
   },
 
   focusContent: function() {
     // Allow Finder listeners to cancel focusing the content.
     for (let l of this._listeners) {
       if (!l.shouldFocusContent())
         return;
     }