Add support for VP8 decode to OmxVideoDecodeAccelerator, for HW that supports it.

media/filters/gpu_video_decoder.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/filters/gpu_video_decoder.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/message_loop.h"
 #include "base/stl_util.h"
@@ skipping 133 matching lines @@
   VideoDecodeAccelerator* vda =
       factories_->CreateVideoDecodeAccelerator(config.profile(), this);
   if (!vda) {
     status_cb.Run(DECODER_ERROR_NOT_SUPPORTED);
     return;
   }
 
   demuxer_stream_ = stream;
   statistics_cb_ = statistics_cb;
 
-  demuxer_stream_->EnableBitstreamConverter();
+  if (config.codec() == kCodecH264)
+    demuxer_stream_->EnableBitstreamConverter();

[scherkus (not reviewing), 2012/07/20 20:20:20]

q: do you recall the reason for having the bitstream converter owned by the
demuxer vs owned by GpuVD?

I guess it cuts down on code duplication when multiple VDs require bitstream
conversion hm..

[Ami GONE FROM CHROMIUM, 2012/07/20 20:25:20]

The decision long predates my involvement with the team.
When xhwang@ reworked the BSF stuff I thought about this a bit but didn't come
up with an obviously-much-better approach, so stayed quiet.

 
   natural_size_ = config.natural_size();
   config_frame_duration_ = GetFrameDuration(config);
 
   DVLOG(1) << "GpuVideoDecoder::Initialize() succeeded.";
   vda_loop_proxy_->PostTaskAndReply(
       FROM_HERE,
       base::Bind(&GpuVideoDecoder::SetVDA, this, vda),
       base::Bind(status_cb, PIPELINE_OK));
 }
@@ skipping 393 matching lines @@
 
   error_occured_ = true;
 
   if (!pending_read_cb_.is_null()) {
     base::ResetAndReturn(&pending_read_cb_).Run(kDecodeError, NULL);
     return;
   }
 }
 
 }  // namespace media