[Remoting Android] DisplayUpdaterFactory cleanups

remoting/protocol/video_renderer.h

 // Copyright 2016 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_VIDEO_RENDERER_H_
 #define REMOTING_CLIENT_VIDEO_RENDERER_H_
 
 namespace remoting {
+class ClientContext;

[Sergey Ulanov, 2016/06/29 23:02:12]

nit: add empty line here

[Yuwei, 2016/06/29 23:38:03]

Done. Also added one below namespace protocol. Not sure what is the guideline
for adding empty lines?

+
 namespace protocol {
-
 class FrameConsumer;
+class PerformanceTracker;
 class SessionConfig;
 class VideoStub;
 
 // VideoRenderer is responsible for decoding and displaying incoming video
 // stream. This interface is used by ConnectionToHost implementations to
 // render received video frames. ConnectionToHost may feed encoded frames to the
 // VideoStub or decode them and pass decoded frames to the FrameConsumer.
 //
 // TODO(sergeyu): Reconsider this design.
 class VideoRenderer {
  public:
   virtual ~VideoRenderer() {}
 
+  // Initializes the video renderer. This allows the renderer to be initialized
+  // after it is constructed. Returns true if initialization succeeds and false
+  // otherwise. An implementation that doesn't use this function to initialize
+  // should always return true.
+  // |perf_tracker| must outlive the renderer.
+  virtual bool Initialize(const ClientContext& client_context,
+                          protocol::PerformanceTracker* perf_tracker) = 0;
+
   // Configures the renderer with the supplied |config|. This must be called
   // exactly once before video data is supplied to the renderer.
   virtual void OnSessionConfig(const SessionConfig& config) = 0;
 
   // Returns the VideoStub interface of this renderer.
   virtual VideoStub* GetVideoStub() = 0;
 
   // Returns the FrameConsumer interface for this renderer.
   virtual FrameConsumer* GetFrameConsumer() = 0;
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_VIDEO_RENDERER_H_