Sky: Add a Point class.

sky/engine/core/painting/Point.h

 // Copyright 2015 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 SKY_ENGINE_CORE_PAINTING_RECT_H_
-#define SKY_ENGINE_CORE_PAINTING_RECT_H_
+#ifndef SKY_ENGINE_CORE_PAINTING_POINT_H_
+#define SKY_ENGINE_CORE_PAINTING_POINT_H_
 
 #include "dart/runtime/include/dart_api.h"
 #include "sky/engine/tonic/dart_converter.h"
-#include "third_party/skia/include/core/SkRect.h"
+#include "third_party/skia/include/core/SkPoint.h"
 
 namespace blink {
-// Very simple wrapper for SkRect to add a null state.
-class Rect {
+// Very simple wrapper for SkPoint to add a null state.
+class Point {
  public:
-  SkRect sk_rect;
-  bool is_null;
+  SkPoint skPoint;
+  bool isNull;

[eseidel, 2015/05/26 22:21:11]

Why this style change?  Is this Chromium style? (I don't even know..., but I
know we're trying to follow it in new code.)

[Matt Perry, 2015/05/26 22:24:08]

Oh - the old code is Chromium style. I saw Blink style everywhere and assumed we
were trying to use that. I'll revert this part of the change.

[Matt Perry, 2015/05/27 17:55:43]

OK, so I'll keep the .cpp/.h filenames because that's the convention in this
directory. But inside the files I'll use Chromium style, with the exception of
any IDL-generated method names (setFoo, etc.)

 };
 
 template<>
-struct DartConverter<Rect, void> {
-  static Rect FromArgumentsWithNullCheck(Dart_NativeArguments args,
+struct DartConverter<Point, void> {
+  static Point FromArgumentsWithNullCheck(Dart_NativeArguments args,
                                          int index,
                                          Dart_Handle& exception);
 };
 
 } // namespace blink
 
-#endif  // SKY_ENGINE_CORE_PAINTING_RECT_H_
+#endif  // SKY_ENGINE_CORE_PAINTING_POINT_H_