Add related_applications field to manifest parser.

content/public/common/manifest.h

 // 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 <vector>
 
 #include "base/strings/nullable_string16.h"
 #include "content/common/content_export.h"
 #include "third_party/WebKit/public/platform/WebScreenOrientationLockType.h"
 #include "ui/gfx/geometry/size.h"
 #include "url/gurl.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 {
   enum DisplayMode {
     DISPLAY_MODE_UNSPECIFIED,
     DISPLAY_MODE_FULLSCREEN,
     DISPLAY_MODE_STANDALONE,
     DISPLAY_MODE_MINIMAL_UI,
     DISPLAY_MODE_BROWSER
   };
 
+  enum RelatedApplicationPlatform {
+    RELATED_APPLICATION_PLATFORM_UNSPECIFIED,
+    RELATED_APPLICATION_PLATFORM_PLAY,
+  };

[mlamouri (slow - plz ping), 2015/04/10 09:49:44]

Shouldn't we have the embedder decide which platforms are recognized? The spec
do not allow invalid string but doesn't have a list of known values. If another
embedder than Chrome wants to accept values like "itunes" or "windows8", we
should probably allow them to do so, right?

[benwells, 2015/04/15 06:45:22]

OK, makes sense. Done.

+
   // Structure representing an icon as per the Manifest specification, see:
   // http://w3c.github.io/manifest/#dfn-icon-object
   struct CONTENT_EXPORT Icon {
     Icon();
     ~Icon();
 
     // MUST be a valid url. If an icon doesn't have a valid URL, it will not be
     // successfully parsed, thus will not be represented in the Manifest.
     GURL src;
 
     // Null if the parsing failed or the field was not present. The type can be
     // any string and doesn't have to be a valid image MIME type at this point.
     // It is up to the consumer of the object to check if the type matches a
     // supported type.
     base::NullableString16 type;
 
     // Default value is 1.0 if the value is missing or invalid.
     double density;
 
     // Empty if the parsing failed, the field was not present or empty.
     // The special value "any" is represented by gfx::Size(0, 0).
     std::vector<gfx::Size> sizes;
 
     // Default density. Set to 1.0.
     static const double kDefaultDensity;
   };
 
+  // Structure representing a related application.
+  struct CONTENT_EXPORT RelatedApplication {
+    RelatedApplication();
+    ~RelatedApplication();
+
+    // The platform on which the application can be found.
+    RelatedApplicationPlatform platform;
+
+    // URL at which the application can be found. Empty if the parsing failed or
+    // the field was not present.
+    GURL url;
+
+    // An id which is used to represent the application on the platform.
+    base::NullableString16 id;

[mlamouri (slow - plz ping), 2015/04/10 09:49:44]

Could you add a comment saying that if the |id| is null |url| shouldn't and vice
versa?

[benwells, 2015/04/15 06:45:22]

Done.

+  };
+
   Manifest();
   ~Manifest();
 
   // Returns whether this Manifest had no attribute set. A newly created
   // Manifest is always empty.
   bool IsEmpty() const;
 
   // 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;
 
   // Empty if the parsing failed or the field was not present.
   GURL start_url;
 
   // Set to DISPLAY_MODE_UNSPECIFIED if the parsing failed or the field was not
   // present.
   DisplayMode display;
 
   // Set to blink::WebScreenOrientationLockDefault if the parsing failed or the
   // field was not present.
   blink::WebScreenOrientationLockType orientation;
 
   // Empty if the parsing failed, the field was not present, empty or all the
   // icons inside the JSON array were invalid.
   std::vector<Icon> icons;
 
+  // Empty if the parsing failed, the field was not present, empty or all the
+  // applications inside the array were invalid. The order of the array
+  // indicates the priority of the application to use.
+  std::vector<RelatedApplication> related_applications;
+
+  // A boolean that is used as a hint for the user agent to say that related
+  // applications should be preferred over the web application. False if missing
+  // or there is a parsing failure.
+  bool prefer_related_applications;
+
   // This is a proprietary extension of the web Manifest, double-check that it
   // is okay to use this entry.
   // Null if parsing failed or the field was not present.
   base::NullableString16 gcm_sender_id;
 
   // This is a proprietary extension of the web Manifest, double-check that it
   // is okay to use this entry.
   // False if parsing failed or the field was not present.
   bool gcm_user_visible_only;
 
   // Maximum length for all the strings inside the Manifest when it is sent over
   // IPC. The renderer process should truncate the strings before sending the
   // Manifest and the browser process must do the same when receiving it.
   static const size_t kMaxIPCStringLength;
 };
 
 } // namespace content
 
-#endif  // CONTENT_PUBLIC_COMMON_MANIFEST_H_
+#endif // CONTENT_PUBLIC_COMMON_MANIFEST_H_