Chromium Code Reviews| Index: content/browser/media/android/browser_media_session_manager.cc |
| diff --git a/content/browser/media/android/browser_media_session_manager.cc b/content/browser/media/android/browser_media_session_manager.cc |
| index 80d3173aa5533c49516c16eaa3079e3654d6cfbe..a9fcd80e9ef5d34c2783e77e675923e0fcb2bc30 100644 |
| --- a/content/browser/media/android/browser_media_session_manager.cc |
| +++ b/content/browser/media/android/browser_media_session_manager.cc |
| @@ -30,10 +30,11 @@ void BrowserMediaSessionManager::OnDeactivate(int session_id, int request_id) { |
| void BrowserMediaSessionManager::OnSetMetadata( |
| int session_id, |
| - const MediaMetadata& insecure_metadata) { |
| + const base::Optional<MediaMetadata>& insecure_metadata) { |
| // When receiving a MediaMetadata, the browser process can't trust that it is |
| // coming from a known and secure source. It must be processed accordingly. |
| - if (!MediaMetadataSanitizer::CheckSanity(insecure_metadata)) { |
| + if (insecure_metadata && |
|
mlamouri (slow - plz ping)
2016/09/01 17:47:49
Maybe `insecure_metadata.has_value()`
Zhiqiang Zhang (Slow)
2016/09/02 12:11:41
Done.
|
| + !MediaMetadataSanitizer::CheckSanity(insecure_metadata.value())) { |
| render_frame_host_->GetProcess()->ShutdownForBadMessage(); |
| return; |
| } |