Build break fixes for issues found using clang.

Change-Id: Id9e74f03ec27b25cdac22c43394ecc0ec9007d19

media/base/media_sample.cc

@@ -30,6 +30,10 @@ MediaSample::MediaSample(const uint8* data,
     side_data_.assign(side_data, side_data + side_data_size);
 }
 
+MediaSample::MediaSample() : dts_(0), pts_(0),
+                             duration_(0),
+                             is_key_frame_(false) {}
+
 MediaSample::~MediaSample() {}
 
 // static

media/base/media_sample.h

@@ -112,8 +112,7 @@ class MediaSample : public base::RefCountedThreadSafe<MediaSample> {
   }
 
   void set_data(const uint8* data, const size_t data_size) {
-    data_size_ = data_size;
+    data_.assign(data, data + data_size);
-    data_.assign(data, data + data_size_);
   }
 
   void set_is_key_frame(bool value) {
@@ -137,10 +136,7 @@ class MediaSample : public base::RefCountedThreadSafe<MediaSample> {
               const uint8* side_data,
               size_t side_data_size,
               bool is_key_frame);
-  MediaSample() : dts_(0), pts_(0),
+  MediaSample();
-                  duration_(0),
-                  is_key_frame_(false),
-                  data_size_(0) {}
   virtual ~MediaSample();
 
   // Decoding time stamp.
@@ -156,7 +152,6 @@ class MediaSample : public base::RefCountedThreadSafe<MediaSample> {
   // http://www.matroska.org/technical/specs/index.html BlockAdditional[A5].
   // Not used by mp4 and other containers.
   std::vector<uint8> side_data_;
-  size_t data_size_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaSample);
 };

media/formats/wvm/wvm_media_parser.cc

@@ -30,7 +30,6 @@ namespace {
   const uint32 kEcmStreamId = 0xF0;
   const uint32 kV2MetadataStreamId = 0xF1;  // EMM_stream
   const uint32 kScramblingBitsMask = 0x30;
-  const uint32 kEncryptedOddKey  = 0x30;
   const uint32 kStartCode1 = 0x00;
   const uint32 kStartCode2 = 0x00;
   const uint32 kStartCode3 = 0x01;
@@ -92,6 +91,8 @@ WvmMediaParser::WvmMediaParser() : is_initialized_(false),
   SHA256_Init(sha_context_);
 }
 
+WvmMediaParser::~WvmMediaParser() {}
+
 void WvmMediaParser::Init(const InitCB& init_cb,
                           const NewSampleCB& new_sample_cb,
                           KeySource* decryption_key_source) {
@@ -891,5 +892,23 @@ void WvmMediaParser::EmitSample(
   }
 }
 
+DemuxStreamIdMediaSample::DemuxStreamIdMediaSample() :
+    demux_stream_id(0),
+    parsed_audio_or_video_stream_id(0) {}
+
+DemuxStreamIdMediaSample::~DemuxStreamIdMediaSample() {}
+
+PrevSampleData::PrevSampleData() {
+  Reset();
+}
+
+PrevSampleData::~PrevSampleData() {}
+
+void PrevSampleData::Reset() {
+  audio_sample = video_sample = NULL;
+  audio_stream_id = video_stream_id = 0;
+  audio_sample_duration = video_sample_duration = 0;
+}
+
 }  // namespace wvm
 }  // namespace media

media/formats/wvm/wvm_media_parser.h

@@ -23,6 +23,9 @@ namespace media {
 namespace wvm {
 
 struct DemuxStreamIdMediaSample {
+ public:
+  DemuxStreamIdMediaSample();
+  ~DemuxStreamIdMediaSample();
   uint32 demux_stream_id;
   uint32 parsed_audio_or_video_stream_id;
   scoped_refptr<MediaSample> media_sample;
@@ -30,12 +33,9 @@ struct DemuxStreamIdMediaSample {
 
 struct PrevSampleData {
  public:
-  PrevSampleData() { Reset(); }
+  PrevSampleData();
-  void Reset() {
+  ~PrevSampleData();
-    audio_sample = video_sample = NULL;
+  void Reset();
-    audio_stream_id = video_stream_id = 0;
-    audio_sample_duration = video_sample_duration = 0;
-  }
   scoped_refptr<MediaSample> audio_sample;
   scoped_refptr<MediaSample> video_sample;
   uint32 audio_stream_id;
@@ -47,7 +47,7 @@ struct PrevSampleData {
 class WvmMediaParser : public MediaParser {
  public:
   WvmMediaParser();
-  virtual ~WvmMediaParser() {}
+  virtual ~WvmMediaParser();
 
   // MediaParser implementation overrides.
   virtual void Init(const InitCB& init_cb,