[tbb-bugs] #8400 [Tor Browser]: Torbutton's browsing history pref seems to require restart

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu Feb 12 15:05:38 UTC 2015

#8400: Torbutton's browsing history pref seems to require restart
     Reporter:           |      Owner:  tbb-team
  mikeperry              |     Status:  needs_revision
         Type:  defect   |  Milestone:
     Priority:  major    |    Version:
    Component:  Tor      |   Keywords:  tbb-rebase-regression, tbb-
  Browser                |  usability, tbb-torbutton, MikePerry201502,
   Resolution:           |  TorBrowserTeam201502
Actual Points:           |  Parent ID:
       Points:           |
Changes (by brade):

 * keywords:
     tbb-rebase-regression, tbb-usability, tbb-torbutton, MikePerry201502,
     tbb-rebase-regression, tbb-usability, tbb-torbutton, MikePerry201502,
 * status:  needs_review => needs_revision


 Mark and I reviewed your fix.  It looks good.  Just a few minor comments:

 - There is an extra blank line after the definition of Cc and Ci.
 - Please use the Cc and Ci constants in the "var sb = ..." statement.
 - You could replace more occurrences of "var" with "let".
 - Remove the extra '.' at the end of your commit message.

Ticket URL: <https://trac.torproject.org/projects/tor/ticket/8400#comment:7>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online

More information about the tbb-bugs mailing list