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

Unified Diff: remoting/client/plugin/pepper_video_renderer_3d.h

Issue 1411283003: Add workaround for PPB_VideoDecoder not setting visible_rect. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
Index: remoting/client/plugin/pepper_video_renderer_3d.h
diff --git a/remoting/client/plugin/pepper_video_renderer_3d.h b/remoting/client/plugin/pepper_video_renderer_3d.h
index 2c229b6a5fa46c9dd264d66691bfd043d371a734..ad7098697f7db72370893e99658034d91d370f3d 100644
--- a/remoting/client/plugin/pepper_video_renderer_3d.h
+++ b/remoting/client/plugin/pepper_video_renderer_3d.h
@@ -98,6 +98,12 @@ class PepperVideoRenderer3D : public PepperVideoRenderer,
webrtc::DesktopVector frame_dpi_;
scoped_ptr<webrtc::DesktopRegion> frame_shape_;
+ // Sizes for the frames in the pipe. Necessary to workaround
+ // crbug.com/542945 . Set only when decoder returns frames that don't have
+ // visible_rect set.
+ typedef std::map<uint32_t, webrtc::DesktopSize> FrameSizesMap;
+ scoped_ptr<FrameSizesMap> frame_sizes_;
Wez 2015/10/19 21:56:16 Do you need something this complicated? Frame size
Sergey Ulanov 2015/10/19 22:19:51 Yes, good point. Done.
+
webrtc::DesktopSize view_size_;
bool initialization_finished_ = false;
« 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