Bug 23584

Summary:

Update application cache code for the latest round of spec changes

Product:

WebKit

Reporter:

Alexey Proskuryakov <ap>

Component:

WebCore Misc.

Assignee:

Alexey Proskuryakov <ap>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
proposed patch	andersca: review+

Description Alexey Proskuryakov 2009-01-28 05:02:24 PST

The changes mostly affect the update process - events are sent at different time, and caches are associated with documents differently.

Comment 1 Alexey Proskuryakov 2009-01-28 05:30:20 PST

Created attachment 27102 [details]
proposed patch

Comment 2 Anders Carlsson 2009-01-28 10:49:28 PST

Comment on attachment 27102 [details]
proposed patch


> +    it = m_pendingMasterResourceLoaders.find(loader);
> +    if (it != m_pendingMasterResourceLoaders.end())
> +        m_pendingMasterResourceLoaders.remove(it);
>  

This could just be m_pendingMasterResourceLoaders.remove(loader);

Looks great otherwise! r=me

Comment 3 Alexey Proskuryakov 2009-01-29 00:27:19 PST

Committed revision 40354.