[tor-bugs] #20680 [Applications/Tor Browser]: Rebase Tor Browser patches to 52 ESR
Tor Bug Tracker & Wiki
blackhole at torproject.org
Mon Apr 3 15:44:01 UTC 2017
#20680: Rebase Tor Browser patches to 52 ESR
-------------------------------------------------+-------------------------
Reporter: arthuredelstein | Owner: tbb-
| team
Type: defect | Status: new
Priority: Medium | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Normal | Resolution:
Keywords: ff52-esr, tbb-7.0-must, | Actual Points:
TorBrowserTeam201703, tbb-7.0-must-nightly |
Parent ID: | Points:
Reviewer: | Sponsor:
| Sponsor4
-------------------------------------------------+-------------------------
Comment (by mcs):
Replying to [comment:35 gk]:
> Here is the next batch:
>
> `43d4fc730a6216433c059345111c5fe9d11deae6`: looks good to me. mcs/brade
could you have a second look?
We have a couple of small comments:
* In `dom/base/nsObjectLoadingContent.cpp`, the `SVGs load as documents,
but are their own capability` comment should not be added (that comment
was removed from the Mozilla code because it is no longer applicable).
* In `parser/html/nsHtml5DocumentBuilder.cpp`, the old patch removed the
`NS_ASSERTION(ssle, "Node didn't QI to style.");` statement. It seems like
we still want to remove that since the patched code uses `if (ssle)` to
handle the now expected case where `ssle` is NULL.
Also, we could instead backport the fix that Mozilla landed for Firefox
53: https://bugzilla.mozilla.org/show_bug.cgi?id=1216893
One problem we see with that approach is that the favicon portion is
missing upstream (#18602).
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/20680#comment:39>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list