Adding VP8 Alpha support in Media Source

media/base/decoder_buffer.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/decoder_buffer.h"
 
 #include "base/logging.h"
 #include "media/base/decrypt_config.h"
 
 namespace media {
@@ skipping 15 matching lines @@
   Initialize();
   memcpy(data_.get(), data, size_);
 }
 
 DecoderBuffer::DecoderBuffer(const uint8* data, int size,
                              const uint8* side_data, int side_data_size)
     : size_(size),
       side_data_size_(side_data_size) {
   if (!data) {
     CHECK_EQ(size_, 0);
+    CHECK(!side_data);
     return;
   }
 
   Initialize();
   memcpy(data_.get(), data, size_);
-  memcpy(side_data_.get(), side_data, side_data_size_);
+  if (side_data_size_ > 0)

[acolwell GONE FROM CHROMIUM, 2013/05/21 19:47:48]

nit: Per offline discussion. s/side_data_size_/side_data/ since that is the real
reason we can't call memcpy.

[vignesh, 2013/05/21 21:05:21]

Done.

+    memcpy(side_data_.get(), side_data, side_data_size_);
 }
 
 DecoderBuffer::~DecoderBuffer() {}
 
 void DecoderBuffer::Initialize() {
   CHECK_GE(size_, 0);
   data_.reset(reinterpret_cast<uint8*>(
       base::AlignedAlloc(size_ + kPaddingSize, kAlignmentSize)));
   memset(data_.get() + size_, 0, kPaddingSize);
   if (side_data_size_ > 0) {
@@ skipping 95 matching lines @@
   std::ostringstream s;
   s << "timestamp: " << timestamp_.InMicroseconds()
     << " duration: " << duration_.InMicroseconds()
     << " size: " << size_
     << " side_data_size: " << side_data_size_
     << " encrypted: " << (decrypt_config_ != NULL);
   return s.str();
 }
 
 }  // namespace media