[Chromoting] Initial plumbing for cursor shape.

remoting/host/capturer.h

Index: remoting/host/capturer.h
diff --git a/remoting/host/capturer.h b/remoting/host/capturer.h
index 5379e7ddc0285ee7683df3c0c3cfeac824b16670..bf944fb7f1fd91cd2189b366b2f63112f576b7a3 100644
--- a/remoting/host/capturer.h
+++ b/remoting/host/capturer.h
@@ -7,11 +7,14 @@
 
 #include "base/basictypes.h"
 #include "base/callback.h"
-#include "remoting/base/capture_data.h"
+#include "media/base/video_frame.h"
 #include "third_party/skia/include/core/SkRegion.h"
 
 namespace remoting {
 
+class CaptureData;
+class CursorShapeData;
+
 // A class to perform the task of capturing the image of a window.
 // The capture action is asynchronous to allow maximum throughput.
 //
@@ -49,6 +52,10 @@ class Capturer {
   typedef base::Callback<void(scoped_refptr<CaptureData>)>
       CaptureCompletedCallback;
 
+  // CursorShapeChangedCallback is called when the cursor shape has changed.
+  typedef base::Callback<void(scoped_refptr<CursorShapeData>)>
+      CursorShapeChangedCallback;

[Wez, 2012/05/23 00:01:57]

If you have Capturers generate protocol::CursorShapeInfo directly then you can
get rid of this callback & the associated setter, and just pass a
CursorShapeCallback directly in to Start(); ScreenRecorder would either
implement that itself, or simply pass the HostControlDispatcher's
implementation.

[garykac, 2012/05/26 01:58:01]

Changed to protocol::CursorShapeInfo and removed setter. Still need callback to
pass into Start().

+
   virtual ~Capturer() {};
 
   // Create platform-specific capturer.
@@ -74,6 +81,10 @@ class Capturer {
   // Called at the end of a capturing session.
   virtual void Stop() = 0;
 
+  // Set callback to be called when the cursor shape changes.
+  virtual void SetCursorShapeChangedCallback(
+      const CursorShapeChangedCallback& callback) = 0;
+
   // Called when the screen configuration is changed.
   virtual void ScreenConfigurationChanged() = 0;