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

[Tor Bug Tracker & Wiki]

#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 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
 https://github.com/arthuredelstein/tor-
 browser/commit/1f1d39cd64da97a1d19857dc27c7b5ba059dd863) is not needed in
 ESR 31 anymore. It can get removed.

 > > 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:36>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online