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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/common/media/media_session_messages_android.h

Issue 2300083002: Wrap MediaMetadata in base::Optional in content and chrome (Closed)
Patch Set: Created 4 years, 3 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 side-by-side diff with in-line comments
Download patch
« content/browser/media/android/browser_media_session_manager.cc ('K') | « content/browser/media/session/media_session.cc ('k') | content/public/browser/web_contents_observer.h » ('j') | content/public/browser/web_contents_observer.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/media/media_session_messages_android.h
diff --git a/content/common/media/media_session_messages_android.h b/content/common/media/media_session_messages_android.h
index 7e951d1344f21f5a1cb2a2ab3a8baa9c7a61bb44..af2b474baf49762194d65456c1585102d51fb5d7 100644
--- a/content/common/media/media_session_messages_android.h
+++ b/content/common/media/media_session_messages_android.h
@@ -5,6 +5,7 @@
// IPC messages for the Media Session API.
// Multiply-included message file, hence no include guard.
+#include "base/optional.h"
#include "content/common/android/gin_java_bridge_errors.h"
#include "content/common/content_export.h"
#include "content/public/common/media_metadata.h"
@@ -41,4 +42,4 @@ IPC_MESSAGE_ROUTED2(MediaSessionHostMsg_Deactivate,
IPC_MESSAGE_ROUTED2(MediaSessionHostMsg_SetMetadata,
int /* request_id*/,
- content::MediaMetadata /* metadata */)
+ base::Optional<content::MediaMetadata> /* metadata */)
« content/browser/media/android/browser_media_session_manager.cc ('K') | « content/browser/media/session/media_session.cc ('k') | content/public/browser/web_contents_observer.h » ('j') | content/public/browser/web_contents_observer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698