appcache content-type check

webkit/browser/appcache/manifest_parser.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // This is a port of ManifestParser.h from WebKit/WebCore/loader/appcache.
 
 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ skipping 36 matching lines @@
   Manifest();
   ~Manifest();
 
   base::hash_set<std::string> explicit_urls;
   NamespaceVector intercept_namespaces;
   NamespaceVector fallback_namespaces;
   NamespaceVector online_whitelist_namespaces;
   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
 
 #endif  // WEBKIT_BROWSER_APPCACHE_MANIFEST_PARSER_H_