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

Unified Diff: media/gpu/avda_codec_allocator.h

Issue 2547743002: media: Remove suffix underscores from public member variables in AVDA (Closed)
Patch Set: another one Created 4 years 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/gpu/android_video_decode_accelerator.cc ('k') | media/gpu/avda_codec_allocator.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/gpu/avda_codec_allocator.h
diff --git a/media/gpu/avda_codec_allocator.h b/media/gpu/avda_codec_allocator.h
index ce05ca7fe6ad2e08b30a5768f3bfa4d23aa5d036..180a91acb1e4fd762c772be808def655420f64e8 100644
--- a/media/gpu/avda_codec_allocator.h
+++ b/media/gpu/avda_codec_allocator.h
@@ -48,33 +48,30 @@ class CodecConfig : public base::RefCountedThreadSafe<CodecConfig> {
public:
CodecConfig();
- // Codec type. Used when we configure media codec.
- VideoCodec codec_ = kUnknownVideoCodec;
-
- // Whether encryption scheme requires to use protected surface.
- bool needs_protected_surface_ = false;
+ VideoCodec codec = kUnknownVideoCodec;
// The surface that MediaCodec is configured to output to.
- gl::ScopedJavaSurface surface_;
+ gl::ScopedJavaSurface surface;
+ int surface_id = SurfaceManager::kNoSurfaceID;
- int surface_id_ = SurfaceManager::kNoSurfaceID;
+ // The MediaCrypto that MediaCodec is configured with for an encrypted stream.
+ MediaDrmBridgeCdmContext::JavaObjectPtr media_crypto;
- // The MediaCrypto object is used in the MediaCodec.configure() in case of
- // an encrypted stream.
- MediaDrmBridgeCdmContext::JavaObjectPtr media_crypto_;
+ // Whether the encryption scheme requires us to use a protected surface.
+ bool needs_protected_surface = false;
- // Initial coded size. The actual size might change at any time, so this
+ // The initial coded size. The actual size might change at any time, so this
// is only a hint.
- gfx::Size initial_expected_coded_size_;
+ gfx::Size initial_expected_coded_size;
- // The type of allocation to use for this. We use this to select the right
- // thread for construction / destruction, and to decide if we should
- // restrict the codec to be software only.
- TaskType task_type_;
+ // The type of allocation to use for. We use this to select the right thread
+ // for construction / destruction, and to decide if we should restrict the
+ // codec to be software only.
+ TaskType task_type;
// Codec specific data (SPS and PPS for H264).
- std::vector<uint8_t> csd0_;
- std::vector<uint8_t> csd1_;
+ std::vector<uint8_t> csd0;
+ std::vector<uint8_t> csd1;
protected:
friend class base::RefCountedThreadSafe<CodecConfig>;
« no previous file with comments | « media/gpu/android_video_decode_accelerator.cc ('k') | media/gpu/avda_codec_allocator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698