[tor-bugs] #17446 [Tor Browser]: Canvas image extraction prompt logic

[Tor Bug Tracker & Wiki]

#17446: Canvas image extraction prompt logic
-------------------------------------------------+-------------------------
 Reporter:  arthuredelstein                      |          Owner:  tbb-
     Type:  defect                               |  team
 Priority:  Medium                               |         Status:
Component:  Tor Browser                          |  needs_revision
 Severity:  Normal                               |      Milestone:
 Keywords:  tbb-fingerprinting,                  |        Version:
  PearlCrescent201511R, TorBrowserTeam201511R    |     Resolution:
Parent ID:                                       |  Actual Points:
  Sponsor:                                       |         Points:
-------------------------------------------------+-------------------------

Comment (by gk):

 Replying to [comment:10 gk]:
 > 2) We have
 > {{{
 >   if (!aDocument || !aCx)
 >     return false;
 >
 >   nsPIDOMWindow *win = aDocument->GetWindow();
 > }}}
 > which basically assumes we'll always have a document as otherwise
 `aDocument->GetWindow()` would blow up. Why do we have the `!aCx` check
 then at all?

 That's actually bullshit, sorry for the noise. Some kind of friday-evening
 code-blindness or something. :(

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