[Remoting Android] Refactor CursorShapeStubProxy from JniDisplayHandler

remoting/client/cursor_shape_stub_proxy.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/cursor_shape_stub_proxy.h>
+#include "base/bind.h"
+#include "base/location.h"
+#include "remoting/proto/control.pb.h"
+
+namespace remoting {
+
+CursorShapeStubProxy::CursorShapeStubProxy(
+    base::WeakPtr<protocol::CursorShapeStub> stub,
+    scoped_refptr<base::SingleThreadTaskRunner> task_runner)
+    : stub_(stub), task_runner_(task_runner) {}
+
+CursorShapeStubProxy::~CursorShapeStubProxy() {}
+
+void CursorShapeStubProxy::SetCursorShape(
+    const protocol::CursorShapeInfo& cursor_shape) {
+  DCHECK(task_runner_);

[Sergey Ulanov, 2016/07/27 20:38:01]

Don't need this DCHECK.
operator-> in scoped_refptr<> already asserts the pointer is not null:
https://codesearch.chromium.org/chromium/src/base/memory/ref_counted.h?rcl=0&...

[Yuwei, 2016/07/27 21:01:36]

Done.

+  task_runner_->PostTask(
+      FROM_HERE, base::Bind(&protocol::CursorShapeStub::SetCursorShape, stub_,
+                            cursor_shape));
+}
+
+}  // namespace remoting