Convert fast/css/background-serialize.html test to testharness.js

Convert fast/css/background-serialize.html test to testharness.js

Cleaning fast/css/* tests to use
testharness.js instead of js-test.js.
This will enable to upstream these tests to web-platform-tests.

Committed: https://crrev.com/cb48e0a4f360c8fc01b7ae0078bfb130d2d4b335
Cr-Commit-Position: refs/heads/master@{#410912}

[MuVen, 2016-08-09 13:20:27]

Description was changed from

==========
done

BUG=
==========

to

==========
Convert fast/css/background-serialize.html test to testharness.js

Cleaning fast/css/* tests to use
testharness.js instead of js-test.js.
This will enable to upstream these tests to web-platform-tests.
==========

[MuVen, 2016-08-09 13:20:47]

sataya.m@samsung.com changed reviewers:
+ srirama.m@samsung.com

[MuVen, 2016-08-09 13:21:11]

sataya.m@samsung.com changed reviewers:
- srirama.m@samsung.com

[MuVen, 2016-08-09 13:21:43]

sataya.m@samsung.com changed reviewers:
+ siva.gunturi@samsung.com

[MuVen, 2016-08-09 13:21:57]

sataya.m@samsung.com changed reviewers:
+ srirama.m@samsung.com
- siva.gunturi@samsung.com

[MuVen, 2016-08-09 13:23:00]

PTAL.

Let me know if this approach is okay. Based on this i shall work on other test
cases.

[MuVen, 2016-08-09 13:24:06]

sataya.m@samsung.com changed reviewers:
- srirama.m@samsung.com

[MuVen, 2016-08-09 13:24:53]

sataya.m@samsung.com changed reviewers:
+ dpranke@chromium.org, srirama.m@samsung.com

[MuVen, 2016-08-09 13:24:54]

PTAL

[Srirama, 2016-08-09 13:34:45]

LGTM, but please wait for Dirk's comments.
Also try to group similar test cases from next time and submit 3 or 4 test cases
per CL.

https://codereview.chromium.org/2228733005/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/fast/css/background-serialize.html (right):

https://codereview.chromium.org/2228733005/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/css/background-serialize.html:12:
assert_equals(test('.test { background: none; }'), '.test { background: none;
}');
Remove this line, this is duplicated.

https://codereview.chromium.org/2228733005/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/css/background-serialize.html:30: function
test(text) {
may be rename it to "getStyle" or something like to make it more readable and
distinguish it from the actual testharness test function.

[MuVen, 2016-08-09 13:46:47]

PTAL.

https://codereview.chromium.org/2228733005/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/fast/css/background-serialize.html (right):

https://codereview.chromium.org/2228733005/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/css/background-serialize.html:12:
assert_equals(test('.test { background: none; }'), '.test { background: none;
}');
On 2016/08/09 13:34:45, Srirama wrote:
> Remove this line, this is duplicated.

Done.

https://codereview.chromium.org/2228733005/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/css/background-serialize.html:30: function
test(text) {
On 2016/08/09 13:34:45, Srirama wrote:
> may be rename it to "getStyle" or something like to make it more readable and
> distinguish it from the actual testharness test function.

Done.

[Dirk Pranke, 2016-08-09 22:55:11]

lgtm

[MuVen, 2016-08-10 00:20:50]

The CQ bit was checked by sataya.m@samsung.com

[MuVen, 2016-08-10 00:20:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
srirama.m@samsung.com
Link to the patchset: https://codereview.chromium.org/2228733005/#ps20001
(title: "addressed review comments")

[commit-bot: I haz the power, 2016-08-10 00:21:49]

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, 2016-08-10 01:37:03]

Description was changed from

==========
Convert fast/css/background-serialize.html test to testharness.js

Cleaning fast/css/* tests to use
testharness.js instead of js-test.js.
This will enable to upstream these tests to web-platform-tests.
==========

to

==========
Convert fast/css/background-serialize.html test to testharness.js

Cleaning fast/css/* tests to use
testharness.js instead of js-test.js.
This will enable to upstream these tests to web-platform-tests.
==========

[commit-bot: I haz the power, 2016-08-10 01:37:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-10 01:38:35]

Description was changed from

==========
Convert fast/css/background-serialize.html test to testharness.js

Cleaning fast/css/* tests to use
testharness.js instead of js-test.js.
This will enable to upstream these tests to web-platform-tests.
==========

to

==========
Convert fast/css/background-serialize.html test to testharness.js

Cleaning fast/css/* tests to use
testharness.js instead of js-test.js.
This will enable to upstream these tests to web-platform-tests.

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

[commit-bot: I haz the power, 2016-08-10 01:38:36]

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