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

Unified Diff: content/renderer/media/pepper_platform_video_decoder.h

Issue 235953002: Merge PlatformVideoDecoder into PPB_VideoDecoder_Impl (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 8 months 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 | « content/content_renderer.gypi ('k') | content/renderer/media/pepper_platform_video_decoder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/pepper_platform_video_decoder.h
diff --git a/content/renderer/media/pepper_platform_video_decoder.h b/content/renderer/media/pepper_platform_video_decoder.h
deleted file mode 100644
index 386de8caef23529e98bec1677d123589d5ff46e1..0000000000000000000000000000000000000000
--- a/content/renderer/media/pepper_platform_video_decoder.h
+++ /dev/null
@@ -1,66 +0,0 @@
-// 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.
-
-#ifndef CONTENT_RENDERER_MEDIA_PEPPER_PLATFORM_VIDEO_DECODER_H_
-#define CONTENT_RENDERER_MEDIA_PEPPER_PLATFORM_VIDEO_DECODER_H_
-
-#include <vector>
-
-#include "base/compiler_specific.h"
-#include "base/memory/ref_counted.h"
-#include "base/message_loop/message_loop.h"
-#include "media/video/video_decode_accelerator.h"
-
-namespace content {
-
-class PlatformVideoDecoder : public media::VideoDecodeAccelerator,
- public media::VideoDecodeAccelerator::Client {
- public:
- explicit PlatformVideoDecoder(int32 command_buffer_route_id);
- virtual ~PlatformVideoDecoder();
-
- // PlatformVideoDecoder (a.k.a. VideoDecodeAccelerator) implementation.
- virtual bool Initialize(media::VideoCodecProfile profile,
- media::VideoDecodeAccelerator::Client* client)
- OVERRIDE;
- virtual void Decode(
- const media::BitstreamBuffer& bitstream_buffer) OVERRIDE;
- virtual void AssignPictureBuffers(
- const std::vector<media::PictureBuffer>& buffers) OVERRIDE;
- virtual void ReusePictureBuffer(int32 picture_buffer_id) OVERRIDE;
- virtual void Flush() OVERRIDE;
- virtual void Reset() OVERRIDE;
- virtual void Destroy() OVERRIDE;
-
- // VideoDecodeAccelerator::Client implementation.
- virtual void ProvidePictureBuffers(uint32 requested_num_of_buffers,
- const gfx::Size& dimensions,
- uint32 texture_target) OVERRIDE;
- virtual void PictureReady(const media::Picture& picture) OVERRIDE;
- virtual void DismissPictureBuffer(int32 picture_buffer_id) OVERRIDE;
- virtual void NotifyError(
- media::VideoDecodeAccelerator::Error error) OVERRIDE;
- virtual void NotifyEndOfBitstreamBuffer(int32 bitstream_buffer_id) OVERRIDE;
- virtual void NotifyFlushDone() OVERRIDE;
- virtual void NotifyResetDone() OVERRIDE;
-
- private:
- // Client lifetime must exceed lifetime of this class.
- // TODO(vrk/fischman): We should take another look at the overall
- // arcitecture of PPAPI Video Decode to make sure lifetime/ownership makes
- // sense, including lifetime of this client.
- media::VideoDecodeAccelerator::Client* client_;
-
- // Route ID for the command buffer associated with video decoder's context.
- int32 command_buffer_route_id_;
-
- // Holds a GpuVideoDecodeAcceleratorHost.
- scoped_ptr<media::VideoDecodeAccelerator> decoder_;
-
- DISALLOW_COPY_AND_ASSIGN(PlatformVideoDecoder);
-};
-
-} // namespace content
-
-#endif // CONTENT_RENDERER_MEDIA_PEPPER_PLATFORM_VIDEO_DECODER_H_
« no previous file with comments | « content/content_renderer.gypi ('k') | content/renderer/media/pepper_platform_video_decoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698