[Webkit-unassigned] [Bug 25562] Potential crash after ApplicationCacheStorage::storeNewestCache() fails.

[bugzilla-daemon at webkit.org]

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


andreip at google.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #30880|review?                     |
               Flag|                            |
  Attachment #30880|0                           |1
        is obsolete|                            |
  Attachment #30899|                            |review?
               Flag|                            |




------- Comment #14 from andreip at google.com  2009-06-03 04:16 PDT -------
Created an attachment (id=30899)
 --> (https://bugs.webkit.org/attachment.cgi?id=30899&action=view)
30880: 30702: Fix for 25562 (take 5)

Small improvement: the logger object needs to live inside storeNewestCache() in
order to be able to also catch the (unlikely) case where the "Update the newest
cache in the group" step fails.

Also improved the comments and removed some whitespace changes.


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