[tor-bugs] #16983 [Tor Browser]: Favicon requests not isolated if one opens the tab list dropdown

Tor Bug Tracker & Wiki blackhole at torproject.org
Fri Sep 18 08:43:39 UTC 2015


#16983: Favicon requests not isolated if one opens the tab list dropdown
-------------------------+-------------------------------------------------
     Reporter:           |      Owner:  tbb-team
  someone_else           |     Status:  closed
         Type:  defect   |  Milestone:
     Priority:  major    |    Version:
    Component:  Tor      |   Keywords:  tbb-linkability,
  Browser                |  TorBrowserTeam201509R
   Resolution:  fixed    |  Parent ID:
Actual Points:           |
       Points:           |
-------------------------+-------------------------------------------------
Changes (by gk):

 * status:  needs_review => closed
 * resolution:   => fixed


Comment:

 Replying to [comment:18 arthuredelstein]:
 > Replying to [comment:13 gk]:
 > > This looks good to me. One thing I was wondering about is that
 inheriting `firstparty` is not necessary anymore. What did it (or was
 supposed to do) in the first place? Or, more direct, why did we need the
 inheritance there given that the icon of the tab is actually set
 elsewhere?
 >
 > I implemented inheriting `firstparty` because in both the `tab` and
 `menuitem` case, there is a child `image` element that is responsible for
 downloading and displaying the image. The `image` attributes of `tab` and
 `menuitem` that are set in `tabbrowser.xml` are copied to the `src` of the
 image element. So we also need to propagate the `firstparty` to that child
 element. Inheritance seems the most general way; otherwise we have to
 write code manually to copy the `firstparty` to the child for each case.

 Sorry, I think you got me wrong (and I totally could see that happening
 given my clumsy words :( ). To give you the context to my comment:13, I
 was just wondering about:
 {{{
 -          <xul:image
 xbl:inherits="src=image,fadein,pinned,selected,busy,crashed,firstparty"
 +          <xul:image
 xbl:inherits="src=image,fadein,pinned,selected,busy,crashed"
 }}}

 Anyway, this is commit 0a287d3bcce4d9e75d1f77af355c19b9292a23c0 on tor-
 browser-38.3.0esr-5.5-1. I applied it optimistically to tor-
 browser-38.3.0esr-5.0-2, too (c15e86cd0f082d63d37fddff1efc35a428cb8eca),
 as this is worthwhile to have in a stable release as well.

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


More information about the tor-bugs mailing list