HTML Standard Tracker

Diff (omit for latest revision)
Filter

Short URL: http://html5.org/r/2350

File a bug

SVNBugCommentTime (UTC)
2350[Gecko] [Webkit] [Google Gears] Appcache: Check the manifest again after updating the cache to make sure it hasn't changed.2008-10-17 23:12
Index: source
===================================================================
--- source	(revision 2349)
+++ source	(revision 2350)
@@ -39116,9 +39116,10 @@
     returns another 4xx or 5xx response or equivalent, or there is a
     DNS error, or the connection times out, or the user cancels the
     download, or the parser for manifests fails when checking the
-    magic signature), or if the resource is labeled with a MIME type
-    other than <code title="">text/cache-manifest</code>, then run the
-    <span>cache failure steps</span>.</p>
+    magic signature), or if the server returned a redirect, or if the
+    resource is labeled with a MIME type other than <code
+    title="">text/cache-manifest</code>, then run the <span>cache
+    failure steps</span>.</p>
 
    </li>
 
@@ -39376,13 +39377,36 @@
 
    </li>
 
-   <li><p>Store <var title="">manifest</var> in <var title="">new
-   cache</var>, if it's not there already, and categorize this entry
-   (whether newly added or not) as <span
-   title="concept-appcache-manifest">the manifest</span>.</p></li>
+   <li>
 
+    <p><span>Fetch</span> the resource from <var title="">manifest
+    URL</var> again, and let <var title="">second manifest</var> be
+    that resource.</p>
+
+   </li>
+
    <li>
 
+    <p>If the previous step failed for any reason, or if the fetching
+    attempt involved a redirect, or if <var title="">second
+    manifest</var> and <var title="">manifest</var> are not
+    byte-for-byte identical, then schedule a rerun of the entire
+    algorithm with the same parameters after a short delay, and run
+    the <span>cache failure steps</span>.</p>
+
+   </li>
+
+   <li>
+
+    <p>Otherwise, store <var title="">manifest</var> in <var
+    title="">new cache</var>, if it's not there already, and
+    categorize this entry (whether newly added or not) as <span
+    title="concept-appcache-manifest">the manifest</span>.</p>
+
+   </li>
+
+   <li>
+
     <p>If this is a <span title="concept-appcache-cache">cache
     attempt</span>, then:</p>
 

|