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

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

Issue 2101833002: [Remoting Android] DisplayUpdaterFactory cleanups (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reviewer's Feedback Created 4 years, 6 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_2d.h
diff --git a/remoting/client/plugin/pepper_video_renderer_2d.h b/remoting/client/plugin/pepper_video_renderer_2d.h
index e772998a0d03a90d60ac2dbfc83bab9c954ce579..4e265e2ab1d7dcf5377a1c622a3a4be95f84101d 100644
--- a/remoting/client/plugin/pepper_video_renderer_2d.h
+++ b/remoting/client/plugin/pepper_video_renderer_2d.h
@@ -19,6 +19,7 @@
#include "ppapi/cpp/view.h"
#include "ppapi/utility/completion_callback_factory.h"
#include "remoting/client/plugin/pepper_video_renderer.h"
+#include "remoting/client/software_video_renderer.h"
#include "remoting/protocol/frame_consumer.h"
#include "third_party/webrtc/modules/desktop_capture/desktop_geometry.h"
@@ -44,14 +45,14 @@ class PepperVideoRenderer2D : public PepperVideoRenderer,
~PepperVideoRenderer2D() override;
// PepperVideoRenderer interface.
- bool Initialize(pp::Instance* instance,
- const ClientContext& context,
- EventHandler* event_handler,
- protocol::PerformanceTracker* perf_tracker) override;
+ void SetPepperContext(pp::Instance* instance,
+ EventHandler* event_handler) override;
void OnViewChanged(const pp::View& view) override;
void EnableDebugDirtyRegion(bool enable) override;
// VideoRenderer interface.
+ bool Initialize(const ClientContext& client_context,
+ protocol::PerformanceTracker* perf_tracker) override;
void OnSessionConfig(const protocol::SessionConfig& config) override;
protocol::VideoStub* GetVideoStub() override;
protocol::FrameConsumer* GetFrameConsumer() override;
@@ -74,7 +75,7 @@ class PepperVideoRenderer2D : public PepperVideoRenderer,
pp::Graphics2D graphics2d_;
- std::unique_ptr<SoftwareVideoRenderer> software_video_renderer_;
+ SoftwareVideoRenderer software_video_renderer_;
// View size in output pixels.
webrtc::DesktopSize view_size_;

Powered by Google App Engine
This is Rietveld 408576698