[Remoting Android] Refactor CursorShapeStubProxy from JniDisplayHandler

remoting/client/callback_cursor_shape_stub.cc

+// 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.
+
+#include "remoting/client/callback_cursor_shape_stub.h"
+
+#include "base/bind.h"
+#include "base/location.h"
+#include "remoting/proto/control.pb.h"
+
+namespace remoting {
+
+CallbackCursorShapeStub::CallbackCursorShapeStub(
+    base::Callback<void(const protocol::CursorShapeInfo&)> callback,
+    scoped_refptr<base::SingleThreadTaskRunner> task_runner)
+    : callback_(callback), task_runner_(task_runner) {}
+
+CallbackCursorShapeStub::~CallbackCursorShapeStub() {}
+
+void CallbackCursorShapeStub::SetCursorShape(
+    const protocol::CursorShapeInfo& cursor_shape) {
+  if (task_runner_) {

[Sergey Ulanov, 2016/07/27 18:26:09]

Do you really need to support case when task_runner_ isn't set?

[Yuwei, 2016/07/27 18:37:20]

It's not really being used... I'll drop it...

[Yuwei, 2016/07/27 19:22:27]

Done. Removed.

+    task_runner_->PostTask(FROM_HERE, base::Bind(callback_, cursor_shape));
+  } else {
+    callback_.Run(cursor_shape);
+  }
+}
+
+}  // namespace remoting