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

Side by Side Diff: media/video/capture/win/capability_list_win.h

Issue 558623002: Video capture: Refactor GetBestMatchedFormat from Win to OS independent (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix mac syntax error Created 6 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Windows specific implementation of VideoCaptureDevice. 5 // Windows specific implementation of VideoCaptureDevice.
6 // DirectShow is used for capturing. DirectShow provide its own threads 6 // DirectShow is used for capturing. DirectShow provide its own threads
7 // for capturing. 7 // for capturing.
8 8
9 #ifndef MEDIA_VIDEO_CAPTURE_WIN_CAPABILITY_LIST_WIN_H_ 9 #ifndef MEDIA_VIDEO_CAPTURE_WIN_CAPABILITY_LIST_WIN_H_
10 #define MEDIA_VIDEO_CAPTURE_WIN_CAPABILITY_LIST_WIN_H_ 10 #define MEDIA_VIDEO_CAPTURE_WIN_CAPABILITY_LIST_WIN_H_
11 11
12 #include <list> 12 #include <list>
13 #include <utility>
13 14
14 #include "base/basictypes.h" 15 #include "base/basictypes.h"
15 #include "base/threading/non_thread_safe.h" 16 #include "base/threading/non_thread_safe.h"
16 #include "media/video/capture/video_capture_types.h" 17 #include "media/video/capture/video_capture_types.h"
17 18
18 namespace media { 19 namespace media {
19 20
20 struct VideoCaptureCapabilityWin { 21 typedef std::pair<int, VideoCaptureFormat> VideoCaptureCapabilityWin;
21 explicit VideoCaptureCapabilityWin(int index) 22 typedef std::list<VideoCaptureCapabilityWin> CapabilityList;
22 : stream_index(index),
23 frame_rate_numerator(0),
24 frame_rate_denominator(1) {}
25 int stream_index;
26 // Internally to Media Foundation Api type devices we use rational framerates
27 // so framerates can be properly represented, f.i. 29.971fps= 30000/1001.
28 int frame_rate_numerator;
29 int frame_rate_denominator;
30 VideoCaptureFormat supported_format;
31 };
32 23
33 class CapabilityList : public base::NonThreadSafe { 24 CapabilityList::const_iterator GetBestMatchedFormat(
34 public: 25 const VideoCaptureFormat& requested,
35 CapabilityList(); 26 const CapabilityList& capabilities);
36 ~CapabilityList();
37
38 bool empty() const { return capabilities_.empty(); }
39
40 // Appends an entry to the list.
41 void Add(const VideoCaptureCapabilityWin& capability);
42
43 // Loops through the list of capabilities and returns an index of the best
44 // matching capability. The algorithm prioritizes height, width, frame rate
45 // and color format in that order.
46 const VideoCaptureCapabilityWin& GetBestMatchedFormat(
47 int requested_width,
48 int requested_height,
49 float requested_frame_rate) const;
50
51 private:
52 typedef std::list<VideoCaptureCapabilityWin> Capabilities;
53 Capabilities capabilities_;
54
55 DISALLOW_COPY_AND_ASSIGN(CapabilityList);
56 };
57 27
58 } // namespace media 28 } // namespace media
59 29
60 #endif // MEDIA_VIDEO_CAPTURE_WIN_CAPABILITY_LIST_WIN_H_ 30 #endif // MEDIA_VIDEO_CAPTURE_WIN_CAPABILITY_LIST_WIN_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698