Use resources/testharness.js for geometry-interfaces-dom-matrix test

Use resources/testharness.js for geometry-interfaces-dom-matrix test

This way the test can be shared with other browser vendors and can be uploaded
as part of the W3C test suite. This CL also removes duplicated test cases from
previous version.

BUG=388780
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180879

[zino, 2014-08-11 13:41:10]

Please take a look.

[pdr., 2014-08-25 01:54:08]

https://codereview.chromium.org/460573004/diff/100001/LayoutTests/TestExpecta...
File LayoutTests/TestExpectations (right):

https://codereview.chromium.org/460573004/diff/100001/LayoutTests/TestExpecta...
LayoutTests/TestExpectations:1311: crbug.com/388780
fast/dom/geometry-interfaces-dom-matrix.html [ NeedsRebaseline ]
NeedsRebasline should not be used for reftests (-expected.html).

https://codereview.chromium.org/460573004/diff/100001/LayoutTests/fast/dom/ge...
File LayoutTests/fast/dom/geometry-interfaces-dom-matrix.html (left):

https://codereview.chromium.org/460573004/diff/100001/LayoutTests/fast/dom/ge...
LayoutTests/fast/dom/geometry-interfaces-dom-matrix.html:45:
shouldBe("matrix.a", "matrix.m11");
Why did you remove these tests (e.g., matrix.a == matrix.m11)? If there was a
reason for this, please include it in your change description.

[pdr., 2014-08-25 01:59:01]

On 2014/08/25 01:54:08, pdr wrote:
>
https://codereview.chromium.org/460573004/diff/100001/LayoutTests/TestExpecta...
> File LayoutTests/TestExpectations (right):
> 
>
https://codereview.chromium.org/460573004/diff/100001/LayoutTests/TestExpecta...
> LayoutTests/TestExpectations:1311: crbug.com/388780
> fast/dom/geometry-interfaces-dom-matrix.html [ NeedsRebaseline ]
> NeedsRebasline should not be used for reftests (-expected.html).

Oops--ignore my comment here.

Please upload the new expectations as part of your patch instead of using
NeedsRebaseline. NeedsRebaseline should be used when we need to pull
platform-specific results off the trybots, but for this change there should not
be platform-specific results.

[zino, 2014-08-25 05:43:37]

@pdr,

Please take a look.

I addressed all your comments.
Thank you.

https://codereview.chromium.org/460573004/diff/100001/LayoutTests/fast/dom/ge...
File LayoutTests/fast/dom/geometry-interfaces-dom-matrix.html (left):

https://codereview.chromium.org/460573004/diff/100001/LayoutTests/fast/dom/ge...
LayoutTests/fast/dom/geometry-interfaces-dom-matrix.html:45:
shouldBe("matrix.a", "matrix.m11");
On 2014/08/25 01:54:08, pdr wrote:
> Why did you remove these tests (e.g., matrix.a == matrix.m11)? If there was a
> reason for this, please include it in your change description.

The test cases is still checked in |DOMMatrix attributes test|.
I don't think we need duplicated tests because matrix.a is always the same with
matrix.m11.
So, I updated the change description.

[pdr., 2014-08-25 17:23:02]

On 2014/08/25 05:43:37, zino wrote:
> @pdr,
> 
> Please take a look.
> 
> I addressed all your comments.
> Thank you.
> 
>
https://codereview.chromium.org/460573004/diff/100001/LayoutTests/fast/dom/ge...
> File LayoutTests/fast/dom/geometry-interfaces-dom-matrix.html (left):
> 
>
https://codereview.chromium.org/460573004/diff/100001/LayoutTests/fast/dom/ge...
> LayoutTests/fast/dom/geometry-interfaces-dom-matrix.html:45:
> shouldBe("matrix.a", "matrix.m11");
> On 2014/08/25 01:54:08, pdr wrote:
> > Why did you remove these tests (e.g., matrix.a == matrix.m11)? If there was
a
> > reason for this, please include it in your change description.
> 
> The test cases is still checked in |DOMMatrix attributes test|.
> I don't think we need duplicated tests because matrix.a is always the same
with
> matrix.m11.
> So, I updated the change description.

LGTM

[zino, 2014-08-25 23:48:16]

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

[commit-bot: I haz the power, 2014-08-25 23:49:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jinho.bang@samsung.com/460573004/120001

[commit-bot: I haz the power, 2014-08-26 00:43:16]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/21063)

[commit-bot: I haz the power, 2014-08-26 02:34:44]

Committed patchset #3 (120001) as 180879