75036 - need to flush chrome images after theme switch

Status: VERIFIED DUPLICATE of bug 75181

(Core Graveyard :: Skinability, defect)

Description

[Joe Hewitt (gone)]

To quote Hyatt from an e-mail that just went around:

There does need to be a way to flush all chrome images (or failing that, all
images) from the memory cache when you switch skins.  This was  dropped on the
floor when libpron landed, and now skin switching has regressed.

Comment 1

[hangas]

Adding nsbeta1 keyword since this is important for skin switching.

Comment 2

[gordon]

nsCacheService has methods for evicting all entries for a given device, and 
nsCacheSession has methods for evicting all entries for a given clientID (say, 
ImgLib).

If ImgLib gets notified that chrome images need to be flushed, it can call 
EvictEntries() on it's cache session.

Oops.  I just checked nsCacheSession.cpp, and EvictEntries() is not implemented.  
That will be easy to fix, because the underlying mechanism in nsCacheService and 
nsMemoryCacheDevice is already in place.  I might be able to get this in this 
weekend.

Comment 3

[David Hyatt]

This was broken by the libpr0n landing.  libpr0n needs to fix it by giving me 
an API to call. Over to pav.

Comment 4

[gordon]

I marked this bug as depending on bug 75171 "implement 
nsCacheSession::EvictEntries()".  I have a proposed patch in that bug, and I'm 
just waiting for review and superreview.

Comment 5

[Stuart Parmenter]

fixed.  dup of 75181

*** This bug has been marked as a duplicate of 75181 ***

Comment 6

[[e-mail] (gone - use [e-mail] instead)]

verified dup.