[Remoting Android] DisplayUpdaterFactory cleanups

remoting/client/software_video_renderer.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_CLIENT_SOFTWARE_VIDEO_RENDERER_H_
 #define REMOTING_CLIENT_SOFTWARE_VIDEO_RENDERER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 29 matching lines @@
 class SoftwareVideoRenderer : public protocol::VideoRenderer,
                               public protocol::VideoStub {
  public:
   // All methods must be called on the same thread the renderer is created. The
   // |decode_task_runner_| is used to decode the video packets. |perf_tracker|
   // must outlive the renderer. |perf_tracker| may be nullptr, performance
   // tracking is disabled in that case.
   SoftwareVideoRenderer(
       scoped_refptr<base::SingleThreadTaskRunner> decode_task_runner,
       protocol::FrameConsumer* consumer,
       protocol::PerformanceTracker* perf_tracker);

[Sergey Ulanov, 2016/06/28 19:08:01]

remove decode_task_runner and perf_tracker, since now they are passed to
Initialize()

[Yuwei, 2016/06/28 21:43:03]

Done. Looks like SoftwareVideoRenderer is used in several places and also in the
iOS repo so I left a legacy constructor here.

   ~SoftwareVideoRenderer() 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;
 
   // protocol::VideoStub interface.
   void ProcessVideoPacket(std::unique_ptr<VideoPacket> packet,
                           const base::Closure& done) override;
 
  private:
   void RenderFrame(std::unique_ptr<protocol::FrameStats> stats,
@@ skipping 14 matching lines @@
   base::ThreadChecker thread_checker_;
 
   base::WeakPtrFactory<SoftwareVideoRenderer> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SoftwareVideoRenderer);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_SOFTWARE_VIDEO_RENDERER_H_