[tor-bugs] #19273 [Applications/Tor Browser]: Write C++ patch to replace external applications helper dialog workaround

[Tor Bug Tracker & Wiki]

#19273: Write C++ patch to replace external applications helper dialog workaround
--------------------------------------+------------------------------
 Reporter:  gk                        |          Owner:  mcs
     Type:  task                      |         Status:  needs_review
 Priority:  High                      |      Milestone:
Component:  Applications/Tor Browser  |        Version:
 Severity:  Normal                    |     Resolution:
 Keywords:  TorBrowserTeam201607R     |  Actual Points:
Parent ID:                            |         Points:
 Reviewer:                            |        Sponsor:  SponsorU
--------------------------------------+------------------------------

Comment (by arthuredelstein):

 I have looked over the code and I think it looks good. (I am still
 building it and will report if I see any problems.)

 General question: do you think a fix for
 https://bugzilla.mozilla.org/show_bug.cgi?id=440892 would use this
 observer mechanism? Or could a proper check for "network.protocol-handler
 .warn-external" be simply built directly into the code? I'm wondering what
 the right approach would be for future upstreaming.

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