Implement MediaMetadata artwork in content

content/public/common/media_metadata.h

Index: content/public/common/media_metadata.h
diff --git a/content/public/common/media_metadata.h b/content/public/common/media_metadata.h
index 3d7f13c59366caac20a29ed98b79c0d8a8478d58..a60e161ead2ac4779b8affc0325847c4c7f548b2 100644
--- a/content/public/common/media_metadata.h
+++ b/content/public/common/media_metadata.h
@@ -5,20 +5,32 @@
 #ifndef CONTENT_PUBLIC_COMMON_MEDIA_METADATA_H_
 #define CONTENT_PUBLIC_COMMON_MEDIA_METADATA_H_
 
+#include <vector>
+
+#include "base/optional.h"
 #include "base/strings/string16.h"
 #include "content/common/content_export.h"
+#include "content/public/common/manifest.h"
 
 namespace content {
 
 // The MediaMetadata is a structure carrying information associated to a
 // content::MediaSession.
 struct CONTENT_EXPORT MediaMetadata {
+  // TODO(zqzhang): move |Manifest::Icon| to a common place. See
+  // https://crbug.com/621859.
+  using Artwork = Manifest::Icon;
+
   MediaMetadata();
   ~MediaMetadata();
 
   bool operator==(const MediaMetadata& other) const;
   bool operator!=(const MediaMetadata& other) const;
 
+  // Apply validity checks and sanitize |artwork|. Returns null if
+  // |artwork| does not pass validity checks.
+  static base::Optional<Artwork> SanitizeArtwork(const Artwork& artwork);

[jochen (gone - plz use gerrit), 2016/06/27 08:12:40]

why is this in public? It's only used from with content

[Zhiqiang Zhang (Slow), 2016/06/28 13:57:01]

The MediaMetadata class is used both in content/ and chrome/. Did you mean
moving this method to somewhere else in content/common/?

+
   // Title associated to the MediaSession.
   base::string16 title;
 
@@ -28,11 +40,17 @@ struct CONTENT_EXPORT MediaMetadata {
   // Album associated to the MediaSession.
   base::string16 album;
 
+  std::vector<Artwork> artwork;
+
   // Maximum length for all the strings inside the MediaMetadata when it is sent
   // over IPC. The renderer process should truncate the strings before sending
   // the MediaMetadata and the browser process must do the same when receiving
   // it.
   static const size_t kMaxIPCStringLength;
+
+  // Maximum type length of Artwork, which conforms to RFC 4288
+  // (https://tools.ietf.org/html/rfc4288).
+  static const size_t kMaxArtworkTypeLength;
 };
 
 }  // namespace content