Re: [tor-bugs] #21850 [Applications/Tor Browser]: Update #16940 patch for e10s (about:tbupdate)

["Tor Bug Tracker & Wiki" <blackhole@xxxxxxxxxxxxxx>]

#21850: Update #16940 patch for e10s (about:tbupdate)
-------------------------------------------------+-------------------------
 Reporter:  arthuredelstein                      |          Owner:  tbb-
                                                 |  team
     Type:  defect                               |         Status:
                                                 |  needs_review
 Priority:  Medium                               |      Milestone:
Component:  Applications/Tor Browser             |        Version:
 Severity:  Normal                               |     Resolution:
 Keywords:  ff52-esr, tbb-e10s, tbb-7.0-must,    |  Actual Points:
  TorBrowserTeam201803R                          |
Parent ID:                                       |         Points:
 Reviewer:                                       |        Sponsor:
-------------------------------------------------+-------------------------
Changes (by mcs):

 * status:  needs_information => needs_review


Comment:

 Replying to [comment:20 gk]:
 > Nice work! One thing I don't understand: why do you have
 > {{{
 > +if (AppConstants.TOR_BROWSER_UPDATE) {
 > +  XPCOMUtils.defineLazyModuleGetter(this, "AboutTBUpdate",
 > +    "resource:///modules/AboutTBUpdate.jsm");
 > +}
 > }}}
 > in `browser.js`? It seems you don't use anything from that module there.

 You are correct; that part of the patch is not needed. Here is a revised
 (and rebased) patch:
 https://gitweb.torproject.org/user/brade/tor-
 browser.git/commit/?h=bug21850-02

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

_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs