appcache content-type check

webkit/browser/appcache/manifest_parser.h

Index: webkit/browser/appcache/manifest_parser.h
diff --git a/webkit/browser/appcache/manifest_parser.h b/webkit/browser/appcache/manifest_parser.h
index 44661ef1fa0da3a65965c728e2f4f1ae10cbd348..9c32ce8d307cadf33ca6b1482f93962a1319c4bb 100644
--- a/webkit/browser/appcache/manifest_parser.h
+++ b/webkit/browser/appcache/manifest_parser.h
@@ -54,10 +54,12 @@ struct WEBKIT_STORAGE_BROWSER_EXPORT Manifest {
   bool online_whitelist_all;
 };
 
-WEBKIT_STORAGE_BROWSER_EXPORT bool ParseManifest(const GURL& manifest_url,
-                                         const char* data,
-                                         int length,
-                                         Manifest& manifest);
+WEBKIT_STORAGE_BROWSER_EXPORT bool ParseManifest(
+    const GURL& manifest_url,
+    const char* data,
+    int length,
+    bool allow_intercepts,

[darin (slow to review), 2014/05/12 23:45:32]

nit: I'm a fan of using enums in cases like these so the callsites are more
readable.

  ParseManifest(..., true, ...);
  ParseManifest(..., false, ...);

versus:

  ParseManifest(..., PARSE_MANIFEST_ALLOWING_INTERCEPTS, ...);
  ParseManifest(..., PARSE_MANIFEST_PER_STANDARD, ...);

+    Manifest& manifest);
 
 }  // namespace appcache