[webkit-reviews] review granted: [Bug 23316] Application cache updating always fails with an assertion : [Attachment 26706] proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 14 09:24:25 PST 2009

Darin Adler <darin at apple.com> has granted Alexey Proskuryakov <ap at webkit.org>'s
request for review:
Bug 23316: Application cache updating always fails with an assertion

Attachment 26706: proposed patch

------- Additional Comments from Darin Adler <darin at apple.com>
> +    // Release our reference to the newest cache. This could cause us to be
> +    // Any ongoing updates will be stopped from destructor.
> +    m_savedNewestCachePointer = m_newestCache.release().get();

I think it would be clearer to write this like this:

    m_savedNewestCachePointer = m_newestCache.get();

That's because release() is a function used to transfer ownership, and we're
really not doing that.


More information about the webkit-reviews mailing list