Implement ManifestParser in content/.

content/public/common/manifest.h

Index: content/public/common/manifest.h
diff --git a/content/public/common/manifest.h b/content/public/common/manifest.h
new file mode 100644
index 0000000000000000000000000000000000000000..e824a47bba65eeaf7bd42932b8dd9f845289d6c3
--- /dev/null
+++ b/content/public/common/manifest.h
@@ -0,0 +1,30 @@
+// Copyright 2014 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.
+
+#ifndef CONTENT_PUBLIC_COMMON_MANIFEST_H_
+#define CONTENT_PUBLIC_COMMON_MANIFEST_H_
+
+#include "base/strings/nullable_string16.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+// The Manifest structure is an internal representation of the Manifest file
+// described in the "Manifest for Web Application" document:
+// http://w3c.github.io/manifest/
+struct CONTENT_EXPORT Manifest {
+  // Returns whether this Manifest had no attribute set. A newly created
+  // Manifest is always empty.
+  bool IsEmpty() const;

[jochen (gone - plz use gerrit), 2014/09/10 08:05:21]

no methods on structs. if you want this to be a struct, it should have a ctor
anyway

[mlamouri (slow - plz ping), 2014/09/10 14:23:43]

What are the alternatives here? I keep struct and make IsEmpty a static method
that takes a Manifest as parameter? or would that be preferable to move to a
class, in which case I will need to have getters and setters, right?

+
+  // Null if the parsing failed or the field was not present.
+  base::NullableString16 name;
+
+  // Null if the parsing failed or the field was not present.
+  base::NullableString16 short_name;
+};
+
+} // namespace content
+
+#endif // CONTENT_PUBLIC_COMMON_MANIFEST_H_