[tbb-bugs] #10760 [Applications/Tor Browser]: Integrate TorButton to TorBrowser core to prevent users from disabling it
Tor Bug Tracker & Wiki
blackhole at torproject.org
Wed May 8 14:35:27 UTC 2019
#10760: Integrate TorButton to TorBrowser core to prevent users from disabling it
Reporter: Rezonansowy | Owner: tbb-
Type: defect | Status: new
Priority: High | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Normal | Resolution:
Keywords: AffectsTails, tbb-parity, | Actual Points:
TorBrowserTeam201904, GeorgKoppen201904 |
Parent ID: #24855 | Points:
Reviewer: | Sponsor:
Changes (by gk):
* status: needs_information => new
Replying to [comment:29 acat]:
> Are there any issues of using in desktop the same
`toolkit/torproject/torbutton` used already in Android (#25013)?
Not really issues, see below.
> Here https://github.com/acatarineu/tor-browser/commit/10760 I took the
patch from #28044 (https://gitweb.torproject.org/user/brade/tor-
and did the same for torbutton, seems to work fine too. Note that `tor-
launcher` patch uses the path `browser/extensions/tor-launcher`, while
current Android and also this one uses `toolkit/torproject/torbutton`. I
realized that in this case the final code goes to `/chrome/torbutton`
inside `omni.ja` instead of `browser/omni.ja` as tor-launcher. Is one path
preferred to the other?
I think the reason Tor Launcher is in the `browser` dir is that we don't
have a Tor Launcher on Android (and won't have one), thus it is desktop-
specific. While that's not the case for Torbutton (even though we don't
use the tor-browser version for desktop yet).
> There is this `GetOverrideStringBundle` function in
that would need to be fixed, since the XPI would not be there. But I
wonder if this code is still needed since `general.useragent.locale` pref
is not there anymore (was moved to `intl.locale.requested`).
Good question and I don't know the answer to it right now...
> If this ticket is in the context of ESR68, then of course there is more
work to do. But if it's just `Integrate TorButton to TorBrowser core to
prevent users from disabling it` I think this patch might be good enough
(together with a change in `tor-browser-build`, will do if this approach
is fine). Perhaps we could track porting to ESR68, refactoring common
parts with tor-launcher, etc. in separate tickets. Or make this ticket
include all those, and keep working on this one.
> Also, not sure if the idea is to abandon the `torbutton` git repository
in favour of continuing development in `tor-browser` repo. According to
-integrate-tor-launcher-into-tor-browser.txt the plan for `tor-launcher`
is to keep the repo, would this also be fine for `torbutton`?
This is a thing we wanted to do for a while now. So, it has not been
really ESR68-related. However, I felt we could start integrating the
remaining Torbutton pieces directly into the browser while we are at it
and while we need to make substantial changes to our UI anyway (given that
there are no overlays anymore in esr68). In that sense it *is* an esr69
item and we should make progress as much as we can now so we have less to
fix later on.
Ideally we would not even have Torbutton as an extension and separate repo
anymore at the end of the process, probably "just" some JS modules
somewhere in `toolkit` and the UI in the respective files per platform.
(Hence the proposal idea as this is not just making a git submodule
We should get rid of the onion button in the toolbar, for instance, and
expose the new identity feature as a button instead directly (the update
check in the menu can go, I think, and the other item is a tor launcher
overlay which we need to think about what to do in the still open tor
I am happy to take a step-wise approach here obviously. The obvious choice
would be 1) get it into tor-browser ala #28044, 2) make the things
compatible with esr68 3) get rid of it as an own extension, however, I
hoped we can avoid doing the full 2) and then 3) by doing part (or the
whole!) 3) while doing minimal 2). :)
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/10760#comment:30>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tbb-bugs