[tbb-bugs] #12620 [Tor Browser]: Rebase TBB patches to Firefox 31 and add unit tests

Tor Bug Tracker & Wiki blackhole at torproject.org
Wed Sep 17 18:13:37 UTC 2014

#12620: Rebase TBB patches to Firefox 31 and add unit tests
     Reporter:  gk       |      Owner:  tbb-team
         Type:  task     |     Status:  new
     Priority:  major    |  Milestone:
    Component:  Tor      |    Version:
  Browser                |   Keywords:  ff31-esr, tbb-rebase, tbb-firefox-
   Resolution:           |  patch, TorBrowserTeam201409D
Actual Points:           |  Parent ID:
       Points:           |

Comment (by arthuredelstein):

 Replying to [comment:36 gk]:
 > Replying to [comment:18 arthuredelstein]:
 > > Replying to [comment:17 gk]:
 > > > Now, we can even observe the respective notification in Torbutton to
 be extra sure that no leaks happening (might be a good QA thing...).
 > >
 > > Definitely. Is there a ticket for this?
 > Done: #13180. But in order to use this the blocks starting with
 `PRNetAddr tempAddr;` need to come after notifying the observer. This is
 okay, there won't be DNS proxy bypasses due to it. While I am at it: the
 second `PRNetAddr tempAddr;` got merged wrongly. It has to be included in
 Resolve() and not in AsyncResolve() (there, the first of those blocks is
 already doing its job). The added code in WebSocketChannel.cpp (in
 browser/commit/1f1d39cd64da97a1d19857dc27c7b5ba059dd863) is not needed in
 ESR 31 anymore. It can get removed.

 Oops -- thanks for catching that mistake. I've added a fixup here:

 Please let me know if this looks right to you.

 > > > The only argument for including the patch in ESR 31 I currently can
 come up with is that ws:// is the only protocol currently being tested in
 the unit test. If that is a show-stopper, fair enough (I planned to add
 tests for the remaining non-internal protocols + getting them merged into
 ESR 38).
 > >
 > > Maybe open a ticket for those additional unit tests?
 > Done: #13181.


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

More information about the tbb-bugs mailing list