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

Side by Side Diff: webrtc/media/devices/gtkvideorenderer.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/devices/gdivideorenderer.h ('k') | webrtc/media/engine/webrtcvideocapturer.cc » ('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 // Definition of class GtkVideoRenderer that implements the abstract class 11 // Definition of class GtkVideoRenderer that implements the abstract class
12 // cricket::VideoRenderer via GTK. 12 // cricket::VideoRenderer via GTK.
13 13
14 #ifndef WEBRTC_MEDIA_DEVICES_GTKVIDEORENDERER_H_ 14 #ifndef WEBRTC_MEDIA_DEVICES_GTKVIDEORENDERER_H_
15 #define WEBRTC_MEDIA_DEVICES_GTKVIDEORENDERER_H_ 15 #define WEBRTC_MEDIA_DEVICES_GTKVIDEORENDERER_H_
16 16
17 #include <memory> 17 #include <memory>
18 18
19 #include "webrtc/base/basictypes.h" 19 #include "webrtc/base/basictypes.h"
20 // TODO(nisse): Temporarily; to be replaced with a forward declaration
21 // of webrtc::VideoFrame when dependency on cricket::VideoFrame is deleted.
22 #include "webrtc/media/base/videoframe.h"
20 #include "webrtc/media/base/videosinkinterface.h" 23 #include "webrtc/media/base/videosinkinterface.h"
21 24
22 typedef struct _GtkWidget GtkWidget; // forward declaration, defined in gtk.h 25 typedef struct _GtkWidget GtkWidget; // forward declaration, defined in gtk.h
23 26
24 namespace cricket { 27 namespace cricket {
25 28
26 class VideoFrame;
27
28 class GtkVideoRenderer : public rtc::VideoSinkInterface<VideoFrame> { 29 class GtkVideoRenderer : public rtc::VideoSinkInterface<VideoFrame> {
29 public: 30 public:
30 GtkVideoRenderer(int x, int y); 31 GtkVideoRenderer(int x, int y);
31 virtual ~GtkVideoRenderer(); 32 virtual ~GtkVideoRenderer();
32 33
33 // Implementation of VideoSinkInterface. 34 // Implementation of VideoSinkInterface.
34 void OnFrame(const VideoFrame& frame) override; 35 void OnFrame(const VideoFrame& frame) override;
35 36
36 private: 37 private:
37 bool SetSize(int width, int height); 38 bool SetSize(int width, int height);
(...skipping 11 matching lines...) Expand all
49 int initial_x_; 50 int initial_x_;
50 int initial_y_; 51 int initial_y_;
51 52
52 int width_; 53 int width_;
53 int height_; 54 int height_;
54 }; 55 };
55 56
56 } // namespace cricket 57 } // namespace cricket
57 58
58 #endif // WEBRTC_MEDIA_DEVICES_GTKVIDEORENDERER_H_ 59 #endif // WEBRTC_MEDIA_DEVICES_GTKVIDEORENDERER_H_
OLDNEW
« no previous file with comments | « webrtc/media/devices/gdivideorenderer.h ('k') | webrtc/media/engine/webrtcvideocapturer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698