Use more specific protocol::CursorShapeStub instead of protocol::ClientStub in VideoScheduler.

remoting/host/video_scheduler.cc

 // Copyright (c) 2012 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.
 
 #include "remoting/host/video_scheduler.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop_proxy.h"
 #include "base/stl_util.h"
 #include "base/sys_info.h"
 #include "base/time.h"
 #include "remoting/capturer/capture_data.h"
 #include "remoting/capturer/mouse_cursor_shape.h"
 #include "remoting/capturer/video_frame_capturer.h"
 #include "remoting/proto/control.pb.h"
 #include "remoting/proto/internal.pb.h"
 #include "remoting/proto/video.pb.h"
-#include "remoting/protocol/client_stub.h"
+#include "remoting/protocol/cursor_shape_stub.h"
 #include "remoting/protocol/message_decoder.h"
 #include "remoting/protocol/video_stub.h"
 #include "remoting/protocol/util.h"
 
 namespace remoting {
 
 // Maximum number of frames that can be processed simultaneously.
 // TODO(hclam): Move this value to CaptureScheduler.
 static const int kMaxPendingCaptures = 2;
 
 // static
 scoped_refptr<VideoScheduler> VideoScheduler::Create(
     scoped_refptr<base::SingleThreadTaskRunner> capture_task_runner,
     scoped_refptr<base::SingleThreadTaskRunner> encode_task_runner,
     scoped_refptr<base::SingleThreadTaskRunner> network_task_runner,
     VideoFrameCapturer* capturer,
     scoped_ptr<VideoEncoder> encoder,
-    protocol::ClientStub* client_stub,
+    protocol::CursorShapeStub* cursor_stub,
     protocol::VideoStub* video_stub) {
   DCHECK(network_task_runner->BelongsToCurrentThread());
   DCHECK(capturer);
   DCHECK(encoder);
-  DCHECK(client_stub);
+  DCHECK(cursor_stub);
   DCHECK(video_stub);
 
   scoped_refptr<VideoScheduler> scheduler = new VideoScheduler(
       capture_task_runner, encode_task_runner, network_task_runner,
-      capturer, encoder.Pass(), client_stub, video_stub);
+      capturer, encoder.Pass(), cursor_stub, video_stub);
   capture_task_runner->PostTask(
       FROM_HERE, base::Bind(&VideoScheduler::StartOnCaptureThread, scheduler));
 
   return scheduler;
 }
 
 // Public methods --------------------------------------------------------------
 
 void VideoScheduler::OnCaptureCompleted(
     scoped_refptr<CaptureData> capture_data) {
@@ skipping 74 matching lines @@
 }
 
 // Private methods -----------------------------------------------------------
 
 VideoScheduler::VideoScheduler(
     scoped_refptr<base::SingleThreadTaskRunner> capture_task_runner,
     scoped_refptr<base::SingleThreadTaskRunner> encode_task_runner,
     scoped_refptr<base::SingleThreadTaskRunner> network_task_runner,
     VideoFrameCapturer* capturer,
     scoped_ptr<VideoEncoder> encoder,
-    protocol::ClientStub* client_stub,
+    protocol::CursorShapeStub* cursor_stub,
     protocol::VideoStub* video_stub)
     : capture_task_runner_(capture_task_runner),
       encode_task_runner_(encode_task_runner),
       network_task_runner_(network_task_runner),
       capturer_(capturer),
       encoder_(encoder.Pass()),
-      cursor_stub_(client_stub),
+      cursor_stub_(cursor_stub),
       video_stub_(video_stub),
       pending_captures_(0),
       did_skip_frame_(false),
       is_paused_(false),
       sequence_number_(0) {
 }
 
 VideoScheduler::~VideoScheduler() {
 }
 
@@ skipping 152 matching lines @@
 
 void VideoScheduler::StopOnEncodeThread(const base::Closure& done_task) {
   DCHECK(encode_task_runner_->BelongsToCurrentThread());
 
   // This is posted by StopOnCaptureThread, so we know that by the time we
   // process it there are no more encode tasks queued.
   network_task_runner_->PostTask(FROM_HERE, done_task);
 }
 
 }  // namespace remoting