Use IntRect instead of FloatRect for window positions and sizes

Source/core/frame/Window.idl

Index: Source/core/frame/Window.idl
diff --git a/Source/core/frame/Window.idl b/Source/core/frame/Window.idl
index a4120ed7aaf43a28502feb94dae265119c6044f0..5c141c9bdbd550b8e9c5609bdab3ccd3ab7106cb 100644
--- a/Source/core/frame/Window.idl
+++ b/Source/core/frame/Window.idl
@@ -101,10 +101,10 @@
     [RuntimeEnabled=CSSOMSmoothScroll] void scrollTo([Default=Undefined] optional ScrollToOptions scrollToOptions);
     void scroll(double x, double y);
     [RuntimeEnabled=CSSOMSmoothScroll] void scroll([Default=Undefined] optional ScrollToOptions scrollOptions);
-    void moveBy([Default=Undefined] optional float x, [Default=Undefined] optional float y); // FIXME: this should take longs not floats.
-    void moveTo([Default=Undefined] optional float x, [Default=Undefined] optional float y); // FIXME: this should take longs not floats.
-    void resizeBy([Default=Undefined] optional float x, [Default=Undefined] optional float y); // FIXME: this should take longs not floats.
-    void resizeTo([Default=Undefined] optional float width, [Default=Undefined] optional float height); // FIXME: this should take longs not floats.
+    void moveBy(optional long x = 0, optional long y = 0);

[Jens Widell, 2015/01/28 12:29:55]

All of these were previously no-ops when called without arguments. For
moveBy/resizeBy, having zero default values results in them being no-ops, and is
a more convenient way to express it in the IDL.

For moveTo/resizeTo, we need to tell a zero argument and a missing argument
apart to not affect observable behavior. This was previously achieved (sort of)
by having [Default=Undefined] which resulted in NaN argument values. (We do
change behavior for explicit NaN arguments now; they were previously treated as
missing, now they become zero.)

It's possible the observable behavior is also wrong, but I wanted to not change
it in this patch, since I already changed quite a lot of code.

[Mike West, 2015/01/28 13:33:17]

Calling these methods without arguments doesn't make any sense, does it? Could
you check Firefox's/IE's behavior to see if we can easily justify breaking the
existing observable behavior? I'd prefer to do that, honestly, as it would also
remove the boolean weirdness from the internal implementation.

[Jens Widell, 2015/01/28 14:09:23]

Firefox 35: arguments are not optional.
IE 11: arguments default to zero (and a minimum window size enforced).

So I guess it should be fine to go for any of those. Non-optional breaks some
current LayoutTests (aside from the test I've already modified) so would sort of
suggest following IE would be slightly safer.

+    void moveTo(optional long x, optional long y);
+    void resizeBy(optional long x = 0, optional long y = 0);
+    void resizeTo(optional long width, optional long height);
 
     [DoNotCheckSecurity] readonly attribute boolean closed;