Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(612)

Side by Side Diff: content/renderer/media/android/renderer_media_session_manager.cc

Issue 2015433003: Implement MediaMetadata artwork in content (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed Dale's comments Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/media/android/renderer_media_session_manager.h" 5 #include "content/renderer/media/android/renderer_media_session_manager.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "content/common/media/media_session_messages_android.h" 8 #include "content/common/media/media_session_messages_android.h"
9 #include "content/public/common/media_metadata.h" 9 #include "content/public/common/media_metadata.h"
10 #include "content/public/renderer/render_thread.h" 10 #include "content/public/renderer/render_thread.h"
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
67 const MediaMetadata& metadata) { 67 const MediaMetadata& metadata) {
68 // Apply some sanity checks on the MediaMetadata before sending over IPC. 68 // Apply some sanity checks on the MediaMetadata before sending over IPC.
69 MediaMetadata ipc_metadata; 69 MediaMetadata ipc_metadata;
70 ipc_metadata.title = 70 ipc_metadata.title =
71 metadata.title.substr(0, MediaMetadata::kMaxIPCStringLength); 71 metadata.title.substr(0, MediaMetadata::kMaxIPCStringLength);
72 ipc_metadata.artist = 72 ipc_metadata.artist =
73 metadata.artist.substr(0, MediaMetadata::kMaxIPCStringLength); 73 metadata.artist.substr(0, MediaMetadata::kMaxIPCStringLength);
74 ipc_metadata.album = 74 ipc_metadata.album =
75 metadata.album.substr(0, MediaMetadata::kMaxIPCStringLength); 75 metadata.album.substr(0, MediaMetadata::kMaxIPCStringLength);
76 76
77 for (const auto& artwork : metadata.artwork) {
78 auto sanitized_artwork =
79 MediaMetadata::SanitizeArtwork(artwork);
80 if (sanitized_artwork)
81 ipc_metadata.artwork.push_back(sanitized_artwork.value());
82
83 if (ipc_metadata.artwork.size() >= MediaMetadata::kMaxNumberOfArtworkImages)
84 break;
85 }
86
77 Send(new MediaSessionHostMsg_SetMetadata(routing_id(), 87 Send(new MediaSessionHostMsg_SetMetadata(routing_id(),
78 session_id, 88 session_id,
79 ipc_metadata)); 89 ipc_metadata));
80 } 90 }
81 91
82 void RendererMediaSessionManager::OnDidActivate(int request_id, bool success) { 92 void RendererMediaSessionManager::OnDidActivate(int request_id, bool success) {
83 DCHECK(pending_activation_requests_.Lookup(request_id)) << request_id; 93 DCHECK(pending_activation_requests_.Lookup(request_id)) << request_id;
84 blink::WebMediaSessionActivateCallback* callback = 94 blink::WebMediaSessionActivateCallback* callback =
85 pending_activation_requests_.Lookup(request_id); 95 pending_activation_requests_.Lookup(request_id);
86 if (success) { 96 if (success) {
87 callback->onSuccess(); 97 callback->onSuccess();
88 } else { 98 } else {
89 callback->onError( 99 callback->onError(
90 blink::WebMediaSessionError(blink::WebMediaSessionError::Activate)); 100 blink::WebMediaSessionError(blink::WebMediaSessionError::Activate));
91 } 101 }
92 pending_activation_requests_.Remove(request_id); 102 pending_activation_requests_.Remove(request_id);
93 } 103 }
94 104
95 void RendererMediaSessionManager::OnDidDeactivate(int request_id) { 105 void RendererMediaSessionManager::OnDidDeactivate(int request_id) {
96 DCHECK(pending_deactivation_requests_.Lookup(request_id)) << request_id; 106 DCHECK(pending_deactivation_requests_.Lookup(request_id)) << request_id;
97 pending_deactivation_requests_.Lookup(request_id)->onSuccess(); 107 pending_deactivation_requests_.Lookup(request_id)->onSuccess();
98 pending_deactivation_requests_.Remove(request_id); 108 pending_deactivation_requests_.Remove(request_id);
99 } 109 }
100 110
101 } // namespace content 111 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698