Opened 8 years ago

Closed 4 years ago

#6528 closed enhancement (fixed)

Combine cache isolation patches and bind to pref

Reported by: mikeperry Owned by: tbb-team
Priority: High Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: tbb-linkability, tbb-bounty, interview, tbb-firefox-patch
Cc: gk Actual Points:
Parent ID: #10820 Points: 14
Reviewer: Sponsor:


We have two cache isolation patches that we should try to get merged upstream, especially since one of them is a crazy large change to the image cache operation and is likely to generate conflicts in future releases.

However, the patches need work before they can be merged. In addition to the pref, we'll want to make the normal cache isolation built-in, instead of relying on torbutton's stanford-safecache.js and associated observers.

Someone will probably also need to write/updated tests. Bleh.

Getting stuff like this polished and merged is a non-trivial amount of work. At a guess, I'd say around 3 to 4 days worth in total?

Child Tickets

#6539defectclosedmikeperryImage cache isolation causes assert crash in debug builds (and other cases?)
#10319enhancementclosedReimplement cache isolation as a direct C++ patch

Change History (7)

comment:1 Changed 8 years ago by mikeperry

Keywords: tbb-linkability added

comment:2 Changed 8 years ago by mikeperry

Keywords: tbb-bounty added

comment:3 Changed 7 years ago by mikeperry

Keywords: interview added

comment:4 Changed 7 years ago by mikeperry

Parent ID: #10820

comment:5 Changed 6 years ago by erinn

Keywords: tbb-firefox-patch added

comment:6 Changed 6 years ago by erinn

Component: Firefox Patch IssuesTor Browser
Owner: changed from mikeperry to tbb-team

comment:7 Changed 4 years ago by gk

Resolution: fixed
Severity: Normal
Status: newclosed

This is done.

Note: See TracTickets for help on using tickets.