Use webrtc::DesktopCapturer for screen capturer implementation.

media/video/capture/screen/mouse_cursor_shape.h

 // 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.
 
 #ifndef MEDIA_VIDEO_CAPTURE_SCREEN_MOUSE_CURSOR_SHAPE_H_
 #define MEDIA_VIDEO_CAPTURE_SCREEN_MOUSE_CURSOR_SHAPE_H_
 
 #include <string>
 
+#include "base/basictypes.h"
 #include "media/base/media_export.h"
-#include "third_party/skia/include/core/SkPoint.h"
-#include "third_party/skia/include/core/SkSize.h"
+#include "third_party/webrtc/modules/desktop_capture/desktop_geometry.h"
 
 namespace media {
 
 // Type used to return mouse cursor shape from video capturers.
 struct MEDIA_EXPORT MouseCursorShape {
   MouseCursorShape();

[alexeypa (please no reviews), 2013/04/26 21:33:58]

I don't think you need to define the constructor and destructor any more. It is
a structure and compiler generated constructor and destructor will do just fine.

[Sergey Ulanov, 2013/05/07 22:25:50]

That's not related to this CL, and I'd rather keep this CL smaller.

[alexeypa (please no reviews), 2013/05/08 22:24:59]

I think it is related. The CL changes the set of members and therefore - their
initialization code. The initialization code is a part of the constructor's
body. So you are changing the constructor already. And if so, removing it
entirely is even a simpler change.

Feel free to do it as a follow up CL, but I personally think it is just not
worth a separate CL.

[Sergey Ulanov, 2013/05/09 18:49:02]

Done.

   ~MouseCursorShape();
 
   // Size of the cursor in screen pixels.
-  SkISize size;
+  webrtc::DesktopSize size;
 
   // Coordinates of the cursor hotspot relative to upper-left corner.
-  SkIPoint hotspot;
+  webrtc::DesktopVector hotspot;
 
   // Cursor pixmap data in 32-bit BGRA format.
   std::string data;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_CAPTURE_SCREEN_MOUSE_CURSOR_SHAPE_H_