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

Side by Side Diff: remoting/client/plugin/pepper_video_renderer_3d.h

Issue 2690943005: Updating OnPictureReady() method name per CR feedback (Closed)
Patch Set: Created 3 years, 10 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #ifndef REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_ 5 #ifndef REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_
6 #define REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_ 6 #define REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <list> 10 #include <list>
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 64
65 // Callback for pp::VideoDecoder::Initialize(). 65 // Callback for pp::VideoDecoder::Initialize().
66 void OnInitialized(int32_t result); 66 void OnInitialized(int32_t result);
67 67
68 // Passes one picture from |pending_frames_| to the |video_decoder_|. 68 // Passes one picture from |pending_frames_| to the |video_decoder_|.
69 void DecodeNextPacket(); 69 void DecodeNextPacket();
70 70
71 // Callback for pp::VideoDecoder::Decode(). 71 // Callback for pp::VideoDecoder::Decode().
72 void OnDecodeDone(int32_t result); 72 void OnDecodeDone(int32_t result);
73 73
74 // Fetches next picture from the |video_decoder_|. 74 // Fetches next picture from the |video_decoder_| if a decoded frame is ready.
75 void GetNextPicture(); 75 void GetNextPictureIfReady();
76 76
77 // Callback for pp::VideoDecoder::GetPicture(). 77 // Callback for pp::VideoDecoder::GetPicture().
78 void OnPictureReady(int32_t result, PP_VideoPicture picture); 78 void OnPictureReady(int32_t result, PP_VideoPicture picture);
79 79
80 // Copies |next_picture_| to |current_picture_| if |next_picture_| is set and 80 // Copies |next_picture_| to |current_picture_| if |next_picture_| is set and
81 // then renders |current_picture_|. Doesn't do anything if |need_repaint_| is 81 // then renders |current_picture_|. Doesn't do anything if |need_repaint_| is
82 // false. 82 // false.
83 void PaintIfNeeded(); 83 void PaintIfNeeded();
84 84
85 // Callback for pp::Graphics3D::SwapBuffers(). 85 // Callback for pp::Graphics3D::SwapBuffers().
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
167 // The hardware limitation. 167 // The hardware limitation.
168 int gl_max_texture_size_; 168 int gl_max_texture_size_;
169 int gl_max_viewport_size_[2]; 169 int gl_max_viewport_size_[2];
170 170
171 DISALLOW_COPY_AND_ASSIGN(PepperVideoRenderer3D); 171 DISALLOW_COPY_AND_ASSIGN(PepperVideoRenderer3D);
172 }; 172 };
173 173
174 } // namespace remoting 174 } // namespace remoting
175 175
176 #endif // REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_ 176 #endif // REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_3D_H_
OLDNEW
« no previous file with comments | « no previous file | remoting/client/plugin/pepper_video_renderer_3d.cc » ('j') | remoting/client/plugin/pepper_video_renderer_3d.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698