Implement PPAPI VideoDecode out-of-process support

ppapi/shared_impl/video_decoder_impl.cc

+// Copyright (c) 2011 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 "ppapi/shared_impl/video_decoder_impl.h"
+
+#include "base/logging.h"
+#include "gpu/command_buffer/client/gles2_implementation.h"
+#include "ppapi/c/pp_errors.h"
+#include "ppapi/proxy/plugin_resource_tracker.h"
+#include "ppapi/thunk/ppb_context_3d_api.h"
+#include "ppapi/thunk/enter.h"
+#include "webkit/plugins/ppapi/resource_tracker.h"
+
+using ppapi::thunk::PPB_Context3D_API;
+
+namespace ppapi {
+
+VideoDecoderImpl::VideoDecoderImpl(bool in_renderer_process)
+    : context3d_id_(0),
+      in_renderer_process_(in_renderer_process),
+      gles2_impl_(NULL) {
+}
+
+thunk::PPB_VideoDecoder_API* VideoDecoderImpl::AsPPB_VideoDecoder_API() {
+  return this;
+}
+
+VideoDecoderImpl::~VideoDecoderImpl() {
+}
+
+bool VideoDecoderImpl::Init(PP_Resource context3d_id,
+                            const PP_VideoConfigElement* decoder_config) {
+  if (!context3d_id || !decoder_config)
+    return false;
+
+  ppapi::thunk::EnterResourceNoLock<PPB_Context3D_API>
+      enter_context(context3d_id, true);
+  if (!enter_context.succeeded())
+    return false;
+  SetGLES2Impl(enter_context.object()->GetGLES2Impl());
+
+  AddRefResource(context3d_id);
+  context3d_id_ = context3d_id;
+  return true;
+}
+
+void VideoDecoderImpl::Destroy() {
+  context3d_id_ = 0;
+  gles2_impl_ = NULL;
+  UnrefResource(context3d_id_);
+}
+
+void VideoDecoderImpl::SetFlushCallback(PP_CompletionCallback callback) {
+  flush_callback_ = callback;
+}
+
+void VideoDecoderImpl::SetResetCallback(PP_CompletionCallback callback) {
+  reset_callback_ = callback;
+}
+
+void VideoDecoderImpl::AddBitstreamBufferCallback(
+    int32 bitstream_buffer_id, PP_CompletionCallback callback) {
+  DCHECK(bitstream_buffer_callbacks_.insert(

[Ami GONE FROM CHROMIUM, 2011/08/02 00:49:08]

Did you intentionally s/CHECK/DCHECK/ in the move here?

[vrk (LEFT CHROMIUM), 2011/08/03 19:04:30]

noooooooooooooooooooooooooooo

CRAP and this is why my Release build wasn't working! Done and yeah, now Release
is working :P

+      std::make_pair(bitstream_buffer_id, callback)).second);
+}
+
+void VideoDecoderImpl::RunFlushCallback() {
+  if (flush_callback_.func != NULL)

[Ami GONE FROM CHROMIUM, 2011/08/02 00:49:08]

Can this be a DCHECK instead?

[vrk (LEFT CHROMIUM), 2011/08/03 19:04:30]

Yes, done.

+    PP_RunAndClearCompletionCallback(&flush_callback_, PP_OK);
+}
+
+void VideoDecoderImpl::RunResetCallback() {
+  if (reset_callback_.func != NULL)

[Ami GONE FROM CHROMIUM, 2011/08/02 00:49:08]

ditto

[vrk (LEFT CHROMIUM), 2011/08/03 19:04:30]

Done.

+    PP_RunAndClearCompletionCallback(&reset_callback_, PP_OK);
+}
+
+void VideoDecoderImpl::RunBitstreamBufferCallback(int32 bitstream_buffer_id) {
+  CallbackById::iterator it =
+      bitstream_buffer_callbacks_.find(bitstream_buffer_id);
+  DCHECK(it != bitstream_buffer_callbacks_.end());
+  PP_CompletionCallback cc = it->second;
+  bitstream_buffer_callbacks_.erase(it);
+  PP_RunCompletionCallback(&cc, PP_OK);
+}
+
+void VideoDecoderImpl::FlushCommandBuffer() {
+  if (gles2_impl_)
+    gles2_impl_->Flush();
+}
+
+bool VideoDecoderImpl::CopyConfigsToVector(
+    const PP_VideoConfigElement* configs_to_copy,
+    std::vector<int32>* out_configs) {
+  // TODO(vrk): This is assuming PP_VideoAttributeDictionary and
+  // VideoAttributeKey have identical enum values. There is no compiler
+  // assert to guarantee this. We either need to add such asserts or
+  // merge PP_VideoAttributeDictionary and VideoAttributeKey.
+  const PP_VideoConfigElement* current = configs_to_copy;
+  while (current && *current != PP_VIDEOATTR_DICTIONARY_TERMINATOR) {
+    if (!(current + 1))

[Ami GONE FROM CHROMIUM, 2011/08/02 00:49:08]

I think you're missing a * here ;)

[vrk (LEFT CHROMIUM), 2011/08/03 19:04:30]

No, sadly, I am just AN IDIOT! (omg sometimes I think dumb things, I BLAME JAVA)

Fixed to make more sense, though still not that fixed. Comment to add real
fixes.

+      return false;
+
+    out_configs->push_back(static_cast<int32>(*current));

[Ami GONE FROM CHROMIUM, 2011/08/02 00:49:08]

Hopefully you can lose these casts if you switch to using vector<PP_VCE>
everywhere.

[vrk (LEFT CHROMIUM), 2011/08/03 19:04:30]

Done.

+    out_configs->push_back(static_cast<int32>(*(current + 1)));
+    current += 2;
+  }
+  return true;
+}
+
+void VideoDecoderImpl::SetGLES2Impl(

[Ami GONE FROM CHROMIUM, 2011/08/02 00:49:08]

Given this has a single caller, I'd simply in-line it there (where a single 
DCHECK(!gles2_impl_ && !context3d_id_);
guarantees Init() is called no more than once).

[vrk (LEFT CHROMIUM), 2011/08/03 19:04:30]

Done.

+    gpu::gles2::GLES2Implementation* gles2_impl) {
+  DCHECK(!gles2_impl_);
+  gles2_impl_ = gles2_impl;
+}
+
+void VideoDecoderImpl::AddRefResource(PP_Resource resource) {
+  if (in_renderer_process_)
+    ::webkit::ppapi::ResourceTracker::Get()->AddRefResource(resource);

[Ami GONE FROM CHROMIUM, 2011/08/02 00:49:08]

Oh my goodness.
/me weeps

[vrk (LEFT CHROMIUM), 2011/08/03 19:04:30]

I knowwwwwwwwwwwwwwwwww

But the ugliness is expected to go away when brettw DOes his TODO in
tracker_base.h to merge these classes! I added a TODO for us in tracker_base.h
to fix VideoDecoderImpl when that time comes.

+  else
+    ::pp::proxy::PluginResourceTracker::GetInstance()->AddRefResource(resource);
+}
+
+void VideoDecoderImpl::UnrefResource(PP_Resource resource) {
+  if (in_renderer_process_) {
+    ::webkit::ppapi::ResourceTracker::Get()->UnrefResource(resource);
+  } else {
+    ::pp::proxy::PluginResourceTracker::
+        GetInstance()->ReleaseResource(resource);
+  }
+}
+
+}  // namespace ppapi