[webkit-reviews] review requested: [Bug 25562] Potential crash after ApplicationCacheStorage::storeNewestCache() fails. : [Attachment 30899] 30880: 30702: Fix for 25562 (take 5)

[bugzilla-daemon at webkit.org]

Andrei Popescu <andreip at google.com> has asked  for review:
Bug 25562: Potential crash after ApplicationCacheStorage::storeNewestCache()
fails.
https://bugs.webkit.org/show_bug.cgi?id=25562

Attachment 30899: 30880: 30702: Fix for 25562 (take 5)
https://bugs.webkit.org/attachment.cgi?id=30899&action=review

------- Additional Comments from Andrei Popescu <andreip at google.com>
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.