Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].
This patch doesn't break the existing web because DOMPoint is still behind the
runtime flag.

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437
Committed: https://crrev.com/e14a6ca3bed003384db8a69064fbbb9cd2a533d6
Cr-Commit-Position: refs/heads/master@{#441099}

[zino, 2016-12-30 08:04:24]

jinho.bang@samsung.com changed reviewers:
+ jinho.bang@samsung.com

[zino, 2016-12-30 08:04:24]

lgtm with nit.

You will have to update webexposed test before landing this patch.
Also, I would like to add a comment to the CL description that this patch
doesn't break the existing web because DOMPoint is still behind the runtime
flag.

If you want to land this, you should add a owner in core directory and then get
a review from him or her.

https://codereview.chromium.org/2606763002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/dom/DOMPoint.idl (left):

https://codereview.chromium.org/2606763002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/dom/DOMPoint.idl:8: // FIXME: The point argument
is optional in the spec:
nit: Please remove this comments.

[Byoungkwon Ko, 2016-12-30 18:30:58]

gogag2@gmail.com changed reviewers:
+ haraken@chromium.org

[Byoungkwon Ko, 2016-12-30 19:10:48]

I deleted the comment.
Thanks.

[haraken, 2016-12-31 00:29:02]

LGTM

[Byoungkwon Ko, 2016-12-31 01:58:23]

The CQ bit was checked by gogag2@gmail.com

[Byoungkwon Ko, 2016-12-31 01:58:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
jinho.bang@samsung.com
Link to the patchset: https://codereview.chromium.org/2606763002/#ps20001
(title: "Delete DOMPoint constructor using DOMPointInit as parameter.")

[commit-bot: I haz the power, 2016-12-31 01:58:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[zino, 2016-12-31 02:55:35]

The CQ bit was unchecked by jinho.bang@samsung.com

[zino, 2016-12-31 03:03:59]

Please update CL description as per my previous comment.

https://codereview.chromium.org/2606763002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/fast/dom/geometry-interfaces-dom-point.html
(left):

https://codereview.chromium.org/2606763002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/geometry-interfaces-dom-point.html:29:
}, 'DOMPoint constructor with x parameter');
Please remove this test as well.

https://codereview.chromium.org/2606763002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/DOMPoint.idl (left):

https://codereview.chromium.org/2606763002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/DOMPoint.idl:9: //
https://www.w3.org/Bugs/Public/show_bug.cgi?id=28108
Please remove this comment as well.

[Byoungkwon Ko, 2016-12-31 04:03:40]

Description was changed from

==========
Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437
==========

to

==========
Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].
This patch doesn't break the existing web because DOMPoint is still behind the
runtime flag.

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437
==========

[Byoungkwon Ko, 2016-12-31 05:32:04]

Description was changed from

==========
Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].
This patch doesn't break the existing web because DOMPoint is still behind the
runtime flag.

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437
==========

to

==========
Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].
This patch doesn't break the existing web because DOMPoint is still behind the
runtime flag.

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437
==========

[zino, 2017-01-03 06:56:08]

The CQ bit was checked by jinho.bang@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-03 06:56:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-03 08:20:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-03 08:20:30]

Dry run: This issue passed the CQ dry run.

[Byoungkwon Ko, 2017-01-03 08:34:27]

The CQ bit was checked by gogag2@gmail.com

[Byoungkwon Ko, 2017-01-03 08:34:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, jinho.bang@samsung.com
Link to the patchset: https://codereview.chromium.org/2606763002/#ps40001
(title: "Delete DOMPoint constructor using DOMPointInit as parameter.")

[commit-bot: I haz the power, 2017-01-03 08:34:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-03 08:39:54]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483432467734970,
"parent_rev": "d780f3d3a9c0117b7183072412d4c7a3fff2afc3", "commit_rev":
"4aa2e241c81e4e23d81271f8b3e4a4d86febaef1"}

[commit-bot: I haz the power, 2017-01-03 08:40:34]

Description was changed from

==========
Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].
This patch doesn't break the existing web because DOMPoint is still behind the
runtime flag.

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437
==========

to

==========
Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].
This patch doesn't break the existing web because DOMPoint is still behind the
runtime flag.

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437

Review-Url: https://codereview.chromium.org/2606763002
==========

[commit-bot: I haz the power, 2017-01-03 08:40:35]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2017-01-03 08:43:14]

Description was changed from

==========
Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].
This patch doesn't break the existing web because DOMPoint is still behind the
runtime flag.

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437

Review-Url: https://codereview.chromium.org/2606763002
==========

to

==========
Delete DOMPoint constructor using DOMPointInit as parameter.

Delete DOMPoint constructor using DOMPointInit as parameter.
DOMPoint interfaces do not need the constructor anymore as following spec[1].
This patch doesn't break the existing web because DOMPoint is still behind the
runtime flag.

[1] https://drafts.fxtf.org/geometry/#dom-dompoint-dompoint

BUG=675437

Committed: https://crrev.com/e14a6ca3bed003384db8a69064fbbb9cd2a533d6
Cr-Commit-Position: refs/heads/master@{#441099}
==========

[commit-bot: I haz the power, 2017-01-03 08:43:15]

Patchset 3 (id:??) landed as
https://crrev.com/e14a6ca3bed003384db8a69064fbbb9cd2a533d6
Cr-Commit-Position: refs/heads/master@{#441099}

[Byoungkwon Ko, 2017-01-03 08:53:10]

https://codereview.chromium.org/2606763002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/fast/dom/geometry-interfaces-dom-point.html
(left):

https://codereview.chromium.org/2606763002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/geometry-interfaces-dom-point.html:29:
}, 'DOMPoint constructor with x parameter');
On 2016/12/31 03:03:59, zino wrote:
> Please remove this test as well.

Done.

https://codereview.chromium.org/2606763002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/DOMPoint.idl (left):

https://codereview.chromium.org/2606763002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/DOMPoint.idl:9: //
https://www.w3.org/Bugs/Public/show_bug.cgi?id=28108
On 2016/12/31 03:03:59, zino wrote:
> Please remove this comment as well.

Done.