diff options
author | dalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-21 22:15:02 +0000 |
---|---|---|
committer | dalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-21 22:15:02 +0000 |
commit | c709f6236c745f6d2a3f31221e3f654180d569cf (patch) | |
tree | 160c506863e99b2dbc128b718b17de26aea67853 /media/base/video_frame.h | |
parent | 73119f4c70e9b4f07b0f2389d1b003a270bddf89 (diff) | |
download | chromium_src-c709f6236c745f6d2a3f31221e3f654180d569cf.zip chromium_src-c709f6236c745f6d2a3f31221e3f654180d569cf.tar.gz chromium_src-c709f6236c745f6d2a3f31221e3f654180d569cf.tar.bz2 |
Replace several NOTREACHED(), NOTIMPLEMENTED() with DVLOG(1), all values are checked later by (Audio|Video)DecoderConfig.IsValidConfig()
Unifies frame configuration check between VideoFrame and VideoDecoderConfig.
BUG=none
TEST=unittests, regression tests.
Review URL: http://codereview.chromium.org/9358013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122886 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/base/video_frame.h')
-rw-r--r-- | media/base/video_frame.h | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/media/base/video_frame.h b/media/base/video_frame.h index e9e95fd..a95dd459 100644 --- a/media/base/video_frame.h +++ b/media/base/video_frame.h @@ -1,4 +1,4 @@ -// Copyright (c) 2011 The Chromium Authors. All rights reserved. +// 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. @@ -51,6 +51,13 @@ class MEDIA_EXPORT VideoFrame : public StreamSample { base::TimeDelta timestamp, base::TimeDelta duration); + // Call prior to CreateFrame to ensure validity of frame configuration. Called + // automatically by VideoDecoderConfig::IsValidConfig(). + static bool IsValidConfig( + Format format, + size_t width, + size_t height); + // Wraps a native texture of the given parameters with a VideoFrame. When the // frame is destroyed |no_longer_needed.Run()| will be called. static scoped_refptr<VideoFrame> WrapNativeTexture( |