[Remoting Android] DisplayUpdaterFactory cleanups

remoting/client/plugin/pepper_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_PLUGIN_PEPPER_VIDEO_RENDERER_H_
 #define REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_H_
 
 #include "remoting/protocol/video_renderer.h"
 
 namespace webrtc {
@@ skipping 30 matching lines @@
     virtual void OnVideoFirstFrameReceived() = 0;
 
     // Called with each frame's updated region, if EnableDebugDirtyRegion(true)
     // was called.
     virtual void OnVideoFrameDirtyRegion(
         const webrtc::DesktopRegion& dirty_region) = 0;
   };
 
   // Initializes the renderer. |instance| and |event_handler| must outlive the
   // renderer. Returns false if the renderer cannot be initialized.
-  virtual bool Initialize(pp::Instance* instance,
-                          const ClientContext& context,
-                          EventHandler* event_handler,
-                          protocol::PerformanceTracker* perf_tracker) = 0;
+  virtual bool InitializePepper(pp::Instance* instance,

[Sergey Ulanov, 2016/06/28 17:59:50]

This now overlaps with Initialize(). Remove |context| and |perf_tracker|
arguments from this methods. Then ChromotingInstance will need to call both
InitializePepper() and then Initialize()
Also maybe better to call it SetPepperContext() or something like that, to make
the purpose clearer.

[Yuwei, 2016/06/28 18:46:06]

Done.

+                                const ClientContext& context,
+                                EventHandler* event_handler,
+                                protocol::PerformanceTracker* perf_tracker) = 0;
 
   // Must be called whenever the plugin view changes.
   virtual void OnViewChanged(const pp::View& view) = 0;
 
   // Enables or disables delivery of dirty region information to the
   // EventHandler, for debugging purposes.
   virtual void EnableDebugDirtyRegion(bool enable) = 0;
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_PLUGIN_PEPPER_VIDEO_RENDERER_H_