Refactored pixel format resize operations in media/video/capture into a function called VideoCaptu…

media/video/capture/video_capture_types.h

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..e4233f9577cbfa8be0b5279e76ae732854b31976 100644
--- a/media/video/capture/video_capture_types.h
+++ b/media/video/capture/video_capture_types.h
@@ -67,6 +67,10 @@ class MEDIA_EXPORT VideoCaptureFormat {
   std::string ToString() const;
   static std::string PixelFormatToString(VideoPixelFormat format);
 
+  // Returns the required buffer size to hold an image of a given
+  // VideoCaptureFormat, with no padding and tightly packed if applicable.
+  size_t ImageAllocationSize() const;

[mcasas, 2015/02/04 23:02:39]

I'm not totally in love with this name, despite being
the same in VideoFrame [1]. Please make a bug saying
that someone sometime should think about a better naming.
Hint: make the TODO look like [2].


[1]
https://code.google.com/p/chromium/codesearch#chromium/src/media/base/video_f...
[2]
https://code.google.com/p/chromium/codesearch#chromium/src/media/video/captur...

+
   // Checks that all values are in the expected range. All limits are specified
   // in media::Limits.
   bool IsValid() const;