Skip to content

Commit

Permalink
fix: Don't filter tabs on browser restart.
Browse files Browse the repository at this point in the history
Call `clearFind` on shutdown as well as uninstall/remove.

Fixes #1015.
Merge pull request #1034 from ericawright/clearFind.
  • Loading branch information
bwinton authored Mar 28, 2017
2 parents 584d47d + e340546 commit bf9fb96
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
4 changes: 4 additions & 0 deletions main.js
Original file line number Diff line number Diff line change
Expand Up @@ -580,6 +580,10 @@ exports.main = function (options, callbacks) {
};

exports.onUnload = function (reason) {
for (let window of browserWindows) {
viewFor(window).VerticalTabs.clearFind();
}

// If the app is shutting down, skip the rest
if (reason === 'shutdown') {
return;
Expand Down
1 change: 0 additions & 1 deletion verticaltabs.js
Original file line number Diff line number Diff line change
Expand Up @@ -1201,7 +1201,6 @@ VerticalTabs.prototype = {
if (tourPanel) {
this.document.getElementById('mainPopupSet').removeChild(tourPanel);
}
this.clearFind();
let urlbar = this.document.getElementById('urlbar');
let url = urlbar.value;
let tabs = this.document.getElementById('tabbrowser-tabs');
Expand Down

0 comments on commit bf9fb96

Please sign in to comment.