Add support for the object-position CSS property.

Source/core/platform/LengthPoint.h

Index: Source/core/platform/LengthPoint.h
diff --git a/Source/core/platform/LengthPoint.h b/Source/core/platform/LengthPoint.h
new file mode 100644
index 0000000000000000000000000000000000000000..0afbd26884bb9db7df4def0e4cd2f10fe713e8e4
--- /dev/null
+++ b/Source/core/platform/LengthPoint.h
@@ -0,0 +1,61 @@
+/*
+ * Copyright (c) 2013, Opera Software ASA. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ * 3. Neither the name of Opera Software ASA nor the names of its
+ *    contributors may be used to endorse or promote products derived
+ *    from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+ * COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT,
+ * INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+ * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
+ * OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef LengthPoint_h
+#define LengthPoint_h
+
+#include "core/platform/Length.h"
+
+namespace WebCore {
+
+struct LengthPoint {

[pdr., 2013/08/27 20:07:19]

Any reason for making this a struct over a class?

[mstensho (USE GERRIT), 2013/08/27 20:23:11]

Only that Length and LengthSize are structs. Personally I would have chosen
class.

+public:
+    LengthPoint() { }

[pdr., 2013/08/27 20:07:19]

Please format this like:
LengthPoint()
{
}

[mstensho (USE GERRIT), 2013/08/29 11:16:12]

Done.

+
+    LengthPoint(Length x, Length y)
+        : m_x(x)
+        , m_y(y) { }

[pdr., 2013/08/27 20:07:19]

Please format this like:

LengthPoint(Length x, Length y)
    : m_x(x)
    , m_y(y)
    {
    }

(Personally, I like how you wrote it better, but we try to follow
http://www.chromium.org/blink/coding-style)

[mstensho (USE GERRIT), 2013/08/29 11:16:12]

Done.

+
+    bool operator==(const LengthPoint& o) const { return m_x == o.m_x && m_y == o.m_y; }
+    bool operator!=(const LengthPoint& o) const { return m_x != o.m_x || m_y != o.m_y; }
+
+    void setX(Length x) { m_x = x; }
+    Length x() const { return m_x; }
+
+    void setY(Length y) { m_y = y; }
+    Length y() const { return m_y; }
+
+private:
+    Length m_x;
+    Length m_y;
+};
+
+} // namespace WebCore
+
+#endif // LengthPoint_h