Implement MediaMetadata artwork in content

content/renderer/media/android/renderer_media_session_manager.cc

 // Copyright 2015 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.
 
 #include "content/renderer/media/android/renderer_media_session_manager.h"
 
 #include "base/logging.h"
 #include "content/common/media/media_session_messages_android.h"
 #include "content/public/common/media_metadata.h"
 #include "content/public/renderer/render_thread.h"
@@ skipping 56 matching lines @@
     const MediaMetadata& metadata) {
   // Apply some sanity checks on the MediaMetadata before sending over IPC.
   MediaMetadata ipc_metadata;
   ipc_metadata.title =
       metadata.title.substr(0, MediaMetadata::kMaxIPCStringLength);
   ipc_metadata.artist =
       metadata.artist.substr(0, MediaMetadata::kMaxIPCStringLength);
   ipc_metadata.album =
       metadata.album.substr(0, MediaMetadata::kMaxIPCStringLength);
 
+  for (const auto& artwork : metadata.artwork) {
+    auto sanitized_artwork =
+        MediaMetadata::SanitizeArtwork(artwork);
+    if (sanitized_artwork)
+      ipc_metadata.artwork.push_back(sanitized_artwork.value());
+
+    if (ipc_metadata.artwork.size() >= MediaMetadata::kMaxNumberOfArtworkImages)

[DaleCurtis, 2016/06/29 17:20:20]

Is this necessary on the renderer side? I.e. it seems to assume an untrusted
browser?

[Zhiqiang Zhang (Slow), 2016/06/29 19:16:13]

I think we should. Otherwise the page may set a super long list of artwork, and
we don't want to send a message that long through IPC.

[DaleCurtis, 2016/07/01 00:01:04]

Ah, this is sending to the browser, not receiving. Gotcha.

+      break;
+  }
+
   Send(new MediaSessionHostMsg_SetMetadata(routing_id(),
                                            session_id,
                                            ipc_metadata));
 }
 
 void RendererMediaSessionManager::OnDidActivate(int request_id, bool success) {
   DCHECK(pending_activation_requests_.Lookup(request_id)) << request_id;
   blink::WebMediaSessionActivateCallback* callback =
       pending_activation_requests_.Lookup(request_id);
   if (success) {
     callback->onSuccess();
   } else {
     callback->onError(
         blink::WebMediaSessionError(blink::WebMediaSessionError::Activate));
   }
   pending_activation_requests_.Remove(request_id);
 }
 
 void RendererMediaSessionManager::OnDidDeactivate(int request_id) {
   DCHECK(pending_deactivation_requests_.Lookup(request_id)) << request_id;
   pending_deactivation_requests_.Lookup(request_id)->onSuccess();
   pending_deactivation_requests_.Remove(request_id);
 }
 
 }  // namespace content