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

Side by Side Diff: webrtc/media/devices/gdivideorenderer.h

Issue 2411953002: Reland of Make cricket::VideoFrame inherit webrtc::VideoFrame. (Closed)
Patch Set: Fix for windows build. Created 4 years, 2 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
« no previous file with comments | « webrtc/media/base/videoframe_unittest.h ('k') | webrtc/media/devices/gtkvideorenderer.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2004 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2004 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 // 11 //
12 // Definition of class GdiVideoRenderer that implements the abstract class 12 // Definition of class GdiVideoRenderer that implements the abstract class
13 // cricket::VideoRenderer via GDI on Windows. 13 // cricket::VideoRenderer via GDI on Windows.
14 14
15 #ifndef WEBRTC_MEDIA_DEVICES_GDIVIDEORENDERER_H_ 15 #ifndef WEBRTC_MEDIA_DEVICES_GDIVIDEORENDERER_H_
16 #define WEBRTC_MEDIA_DEVICES_GDIVIDEORENDERER_H_ 16 #define WEBRTC_MEDIA_DEVICES_GDIVIDEORENDERER_H_
17 #ifdef WIN32 17 #ifdef WIN32
18 18
19 #include <memory> 19 #include <memory>
20 20
21 // TODO(nisse): Temporarily; to be replaced with a forward declaration
22 // of webrtc::VideoFrame when dependency on cricket::VideoFrame is deleted.
23 #include "webrtc/media/base/videoframe.h"
21 #include "webrtc/media/base/videosinkinterface.h" 24 #include "webrtc/media/base/videosinkinterface.h"
22 25
23 namespace cricket { 26 namespace cricket {
24 27
25 class VideoFrame;
26
27 class GdiVideoRenderer : public rtc::VideoSinkInterface<cricket::VideoFrame> { 28 class GdiVideoRenderer : public rtc::VideoSinkInterface<cricket::VideoFrame> {
28 public: 29 public:
29 GdiVideoRenderer(int x, int y); 30 GdiVideoRenderer(int x, int y);
30 virtual ~GdiVideoRenderer(); 31 virtual ~GdiVideoRenderer();
31 32
32 // Implementation of VideoSinkInterface 33 // Implementation of VideoSinkInterface
33 void OnFrame(const VideoFrame& frame) override; 34 void OnFrame(const VideoFrame& frame) override;
34 35
35 private: 36 private:
36 class VideoWindow; // forward declaration, defined in the .cc file 37 class VideoWindow; // forward declaration, defined in the .cc file
37 std::unique_ptr<VideoWindow> window_; 38 std::unique_ptr<VideoWindow> window_;
38 // The initial position of the window. 39 // The initial position of the window.
39 int initial_x_; 40 int initial_x_;
40 int initial_y_; 41 int initial_y_;
41 }; 42 };
42 43
43 } // namespace cricket 44 } // namespace cricket
44 45
45 #endif // WIN32 46 #endif // WIN32
46 #endif // WEBRTC_MEDIA_DEVICES_GDIVIDEORENDERER_H_ 47 #endif // WEBRTC_MEDIA_DEVICES_GDIVIDEORENDERER_H_
OLDNEW
« no previous file with comments | « webrtc/media/base/videoframe_unittest.h ('k') | webrtc/media/devices/gtkvideorenderer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698