[tor-bugs] #10095 [TorBrowserButton]: Setting screen resolution to a multiple of 200 x 100 is not working reliably with patch in #8478

[Tor Bug Tracker & Wiki]

#10095: Setting screen resolution to a multiple of 200 x 100 is not working
reliably with patch in #8478
----------------------------+----------------------------------------------
     Reporter:  gk          |      Owner:  mikeperry
         Type:  defect      |     Status:  needs_review
     Priority:  critical    |  Milestone:
    Component:              |    Version:
  TorBrowserButton          |   Keywords:  tbb-fingerprinting, tbb-testcase
   Resolution:              |  Parent ID:
Actual Points:              |
       Points:              |
----------------------------+----------------------------------------------

Comment (by gk):

 Replying to [comment:25 intrigeri]:
 > Replying to [comment:24 gk]:
 > > Works for me with TAILS (even without the new patch). Thus, I guess
 your issue might be #9268. Could that be the case?
 >
 > I've no idea. Could #9268 account for a 1-pixel offset?

 Yes, happened to me. Could you remove the panel/toolbar at the bottom in
 order to check if that additional space "solves" the problem?

 > > You could test whether the resizing logic works on your machine if you
 replace
 > > {{{
 > >       height = Math.floor(maxHeight/100.0)*100;
 > > }}}
 > > with
 > > {{{
 > >       height = 500;
 > > }}}
 > > in torbutton.js.
 >
 > If I do this, then I get 1000x500 both with and without the new patch.

 Good. So the code is generally working but not in your particular
 situation.

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