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

Unified Diff: media/video/capture/video_capture_types.h

Issue 897483002: Refactored pixel format resize operations in media/video/capture into a function called VideoCaptu… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed operator spacing. Created 5 years, 11 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
Index: media/video/capture/video_capture_types.h
diff --git a/media/video/capture/video_capture_types.h b/media/video/capture/video_capture_types.h
index f8e8cab9ebaa4ed48aee99fd319333249d8a2354..80b0e1108bd0af7603a8b01772393f31f742b43e 100644
--- a/media/video/capture/video_capture_types.h
+++ b/media/video/capture/video_capture_types.h
@@ -67,6 +67,11 @@ class MEDIA_EXPORT VideoCaptureFormat {
std::string ToString() const;
static std::string PixelFormatToString(VideoPixelFormat format);
+ // Returns the minimum required image size in bytes for a VideoCaptureFormat
mcasas 2015/02/04 17:00:02 s/.../required buffer size to hold an image of a g
+ // instance.Return value is the exact space required to tightly pack the data
mcasas 2015/02/04 17:00:02 Blank after .
+ // without adding any padding.
+ size_t ImageAllocationSize() const;
mcasas 2015/02/04 17:00:02 Suggest re-writing like: Returns the required buf
+
// Checks that all values are in the expected range. All limits are specified
// in media::Limits.
bool IsValid() const;

Powered by Google App Engine
This is Rietveld 408576698