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

Unified Diff: media/mojo/common/media_type_converters.cc

Issue 2416183002: media: Use native DemuxerStream enum types in media mojo interfaces (Closed)
Patch Set: comments addressed Created 4 years, 2 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
« no previous file with comments | « media/mojo/clients/mojo_demuxer_stream_impl.cc ('k') | media/mojo/interfaces/demuxer_stream.mojom » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/mojo/common/media_type_converters.cc
diff --git a/media/mojo/common/media_type_converters.cc b/media/mojo/common/media_type_converters.cc
index 1c52a6d3033d0ffd64d2ba2a39033928713f7193..e07e0dc1141d5cd38fdfd2f64ea0349eff0bfb34 100644
--- a/media/mojo/common/media_type_converters.cc
+++ b/media/mojo/common/media_type_converters.cc
@@ -15,14 +15,12 @@
#include "media/base/cdm_key_information.h"
#include "media/base/decoder_buffer.h"
#include "media/base/decrypt_config.h"
-#include "media/base/demuxer_stream.h"
#include "media/base/encryption_scheme.h"
#include "media/base/media_keys.h"
#include "media/base/subsample_entry.h"
#include "media/base/video_decoder_config.h"
#include "media/base/video_frame.h"
#include "media/mojo/common/mojo_shared_buffer_video_frame.h"
-#include "media/mojo/interfaces/demuxer_stream.mojom.h"
#include "mojo/public/cpp/system/buffer.h"
namespace mojo {
@@ -39,34 +37,6 @@ namespace mojo {
media::mojom::mojo_enum_value), \
"Mismatched enum: " #media_enum_value " != " #mojo_enum_value)
-// DemuxerStream Type. Note: Mojo DemuxerStream's don't have the TEXT type.
-ASSERT_ENUM_EQ_RAW(DemuxerStream::Type,
- DemuxerStream::UNKNOWN,
- DemuxerStream::Type::UNKNOWN);
-ASSERT_ENUM_EQ_RAW(DemuxerStream::Type,
- DemuxerStream::AUDIO,
- DemuxerStream::Type::AUDIO);
-ASSERT_ENUM_EQ_RAW(DemuxerStream::Type,
- DemuxerStream::VIDEO,
- DemuxerStream::Type::VIDEO);
-static_assert(
- media::DemuxerStream::NUM_TYPES ==
- static_cast<media::DemuxerStream::Type>(
- static_cast<int>(media::mojom::DemuxerStream::Type::LAST_TYPE) + 2),
- "Mismatched enum: media::DemuxerStream::NUM_TYPES != "
- "media::mojom::DemuxerStream::Type::LAST_TYPE + 2");
-
-// DemuxerStream Status.
-ASSERT_ENUM_EQ_RAW(DemuxerStream::Status,
- DemuxerStream::kOk,
- DemuxerStream::Status::OK);
-ASSERT_ENUM_EQ_RAW(DemuxerStream::Status,
- DemuxerStream::kAborted,
- DemuxerStream::Status::ABORTED);
-ASSERT_ENUM_EQ_RAW(DemuxerStream::Status,
- DemuxerStream::kConfigChanged,
- DemuxerStream::Status::CONFIG_CHANGED);
-
// CipherMode
ASSERT_ENUM_EQ_RAW(EncryptionScheme::CipherMode,
EncryptionScheme::CipherMode::CIPHER_MODE_UNENCRYPTED,
« no previous file with comments | « media/mojo/clients/mojo_demuxer_stream_impl.cc ('k') | media/mojo/interfaces/demuxer_stream.mojom » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698