[Remoting Android] DisplayUpdaterFactory cleanups

remoting/protocol/video_renderer.h

Index: remoting/protocol/video_renderer.h
diff --git a/remoting/protocol/video_renderer.h b/remoting/protocol/video_renderer.h
index 4f0eba9a69a2d513eba579fa102dfffa947bdb29..def9b3e55e9e2b4c67c1a5e80226fb0a31a26bfe 100644
--- a/remoting/protocol/video_renderer.h
+++ b/remoting/protocol/video_renderer.h
@@ -6,9 +6,11 @@
 #define REMOTING_CLIENT_VIDEO_RENDERER_H_
 
 namespace remoting {
-namespace protocol {
+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;
 
@@ -22,6 +24,14 @@ 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;