AppCacheExecutableHandlers - parse manifest file and store the 'bit' indicating executable.

webkit/appcache/appcache_interfaces.cc

Index: webkit/appcache/appcache_interfaces.cc
===================================================================
--- webkit/appcache/appcache_interfaces.cc	(revision 194203)
+++ webkit/appcache/appcache_interfaces.cc	(working copy)
@@ -30,6 +30,8 @@
 const char kHttpGETMethod[] = "GET";
 const char kHttpHEADMethod[] = "HEAD";
 
+const char kEnableExecutableHandlers[] = "enable-appcache-executable-handlers";
+
 const base::FilePath::CharType kAppCacheDatabaseName[] =
     FILE_PATH_LITERAL("Index");
 
@@ -61,7 +63,8 @@
 
 Namespace::Namespace()
     : type(FALLBACK_NAMESPACE),
-      is_pattern(false) {
+      is_pattern(false),
+      is_executable(false) {
 }
 
 Namespace::Namespace(
@@ -69,9 +72,20 @@
     : type(type),
       namespace_url(url),
       target_url(target),
-      is_pattern(is_pattern) {
+      is_pattern(is_pattern),
+      is_executable(false) {

[alecflett, 2013/04/16 22:28:56]

Seems like its worth keeping a single constructor around for explicitly
requiring is_executable = false

[michaeln, 2013/04/16 22:43:02]

thought about it, certainly if/when we decide to keep this exp in the code base
for the long haul would want to do that, but i expect we'll yank it at some
point so didn't do that to make it easier to yank

i could just not have the extra ctor at all and alter the callsite(s) to set the
member after construction

 }
 
+Namespace::Namespace(
+    NamespaceType type, const GURL& url, const GURL& target,
+    bool is_pattern, bool is_executable)
+    : type(type),
+      namespace_url(url),
+      target_url(target),
+      is_pattern(is_pattern),
+      is_executable(is_executable) {
+}
+
 Namespace::~Namespace() {
 }