[WIP] Implement DOMPoint of geometry interface.

[WIP] Implement DOMPoint of geometry interface.

WIP

The specification:
  http://dev.w3.org/fxtf/geometry/

Intent to Implement:
  https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/V_bJNtOg0oM

BUG=388780

[zino, 2014-07-17 13:54:30]

Dear @haraken,

I've just uploaded CL for DOMPoint implementation.
I have some questions about Blink-in-javascript.
Please review and guide.

Thank you :)

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.idl
File Source/core/dom/DOMPoint.idl (right):

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.idl...
Source/core/dom/DOMPoint.idl:16: // FIXME: How can we pass arguments to
constructor?
Question #1: How can we use multiple constructor (which has some arguments)?
I'm not sure but the arguments doesn't seem to be passed to current initialize
function.

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.js
File Source/core/dom/DOMPoint.js (right):

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.js#...
Source/core/dom/DOMPoint.js:12: // FIXME: How can we call parent initialize
function?
QUESTION #2: How can we call parent initialize function?

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.js#...
Source/core/dom/DOMPoint.js:21: get: function() { return this.m_x; },
QUESTION #3: This getter function is duplicated with DOMPointReadOnly.

[haraken, 2014-07-17 15:01:59]

Thanks for working on Blink-in-JS! It looks like Blink-in-JS is missing a couple
of mechanisms to support DOMPoint.

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.idl
File Source/core/dom/DOMPoint.idl (right):

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.idl...
Source/core/dom/DOMPoint.idl:16: // FIXME: How can we pass arguments to
constructor?
On 2014/07/17 13:54:30, zino wrote:
> Question #1: How can we use multiple constructor (which has some arguments)?
> I'm not sure but the arguments doesn't seem to be passed to current initialize
> function.

A short answer is that constructors are not yet implemented in Blink-in-JS and
should be implemented. (I welcome your contribution :-)

initialize() in Blink-in-JS is a different thing from constructors. initialize()
is just a method to initialize members of |this| object. initialize() is called
for |this| object at the first time when any DOM attribute or DOM method of
|this| object is going to be called.

On the other hand, constructors are called when 'new DOMPoint(...)' is called. I
think we need to implement a mechanism to invoke DOMPointPrototype.new(...) when
'new DOMPoint(...)' is called. (I think we should avoid using
DOMPointPrototype.constructor(...) since "constructor" is a built-in property
and will cause unexpected conflict.)

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.idl...
Source/core/dom/DOMPoint.idl:22: [ImplementedInPrivateScript] inherit attribute
unrestricted double x;

Before discussing Blink-in-JS, 'inherit attribute' is not supported in the IDL
compiler (because no real IDL files are using it).

Do we really need to use inherit attributes here? Probably is it worth changing
the spec so that the behavior aligns with other DOM features?

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.js
File Source/core/dom/DOMPoint.js (right):

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.js#...
Source/core/dom/DOMPoint.js:12: // FIXME: How can we call parent initialize
function?
On 2014/07/17 13:54:30, zino wrote:
> QUESTION #2: How can we call parent initialize function?

I think this should be handled by PrivateScriptRunner.cpp. If an interface A
inherits from an interface B and A's object is accessed, PrivateScriptRunner
should call A.initialize() and then B.initialize().

However, the mechanism is not yet implemented. (I'm happy if you implement it
:-)

https://codereview.chromium.org/399923002/diff/1/Source/core/dom/DOMPoint.js#...
Source/core/dom/DOMPoint.js:21: get: function() { return this.m_x; },
On 2014/07/17 13:54:30, zino wrote:
> QUESTION #3: This getter function is duplicated with DOMPointReadOnly.

As mentioned in DOMPoint.idl, inherit attributes are not supported in the IDL
compiler (before considering Blink-in-JS). If we really need it, we need to
implement it.

[abarth-chromium, 2014-07-17 18:00:59]

not lgtm

I don't think we should use Blink-in-JS to implement these geometry interfaces. 
Let's get Blink-in-JS working for legacy cruft like <marquee> and XSLT first.

[zino, 2014-07-18 00:26:46]

On 2014/07/17 18:00:59, abarth wrote:
> not lgtm
> 
> I don't think we should use Blink-in-JS to implement these geometry
interfaces. 
> Let's get Blink-in-JS working for legacy cruft like <marquee> and XSLT first.

That's music to my ears!
I'll start to reimplement the features in C++ sides.