[Webkit-unassigned] [Bug 30209] Cache::m_liveDecodedResource is not sorted by CachedResource::m_lastDecodedAccessTime

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=30209


Holger Freyther <zecke at selfish.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #40868|                            |review?
               Flag|                            |




--- Comment #1 from Holger Freyther <zecke at selfish.org>  2009-10-08 07:32:23 PDT ---
Created an attachment (id=40868)
 --> (https://bugs.webkit.org/attachment.cgi?id=40868)
Update m_lastDecodedAccessTime in CachedResource::setDecodedSize

Update the m_lastDecodedAccessTime when we insert the resource. The code in
CachedImage is checking the FrameView painting time but the CachedResource is
not doing this when using WTF::currentTime so I decided to not do it.

I just recognized that this has a problem with the CachedImage::didDraw code...
as this is using FrameView time which is before ::currentTime(). So
CachedResource::setDecodedSize should use the FrameView time as well.

I'm posting this patch anyway to get feedback on the issue and the proposed
resolution.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.