Convert some flexbox tests to testharness.js

Convert some flexbox tests to testharness.js

R=eae@chromium.org,dgrogan@chromium.org

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

[cbiesinger, 2016-07-07 00:37:48]

The CQ bit was checked by cbiesinger@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-07 00:38:12]

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, 2016-07-07 03:47:40]

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

[commit-bot: I haz the power, 2016-07-07 03:47:41]

Dry run: This issue passed the CQ dry run.

[eae, 2016-07-12 16:56:54]

LGTM w/nit

https://codereview.chromium.org/2130543003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/css3/flexbox/align-absolute-child.html
(right):

https://codereview.chromium.org/2130543003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/css3/flexbox/align-absolute-child.html:148:
test(function() {
Shouldn't the for in loop be inside the test funciton?

[cbiesinger, 2016-07-12 17:46:13]

https://codereview.chromium.org/2130543003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/css3/flexbox/align-absolute-child.html
(right):

https://codereview.chromium.org/2130543003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/css3/flexbox/align-absolute-child.html:148:
test(function() {
On 2016/07/12 16:56:54, eae wrote:
> Shouldn't the for in loop be inside the test funciton?

I liked having one PASS line for each iteration of the loop. But I can change it
if you want.

[eae, 2016-07-12 17:49:56]

Ah, I assumed each assert would get a pass line (like they do for FAIL lines).

Whatever you prefer.

[cbiesinger, 2016-07-12 19:00:24]

The CQ bit was checked by cbiesinger@chromium.org

[cbiesinger, 2016-07-12 19:00:34]

On 2016/07/12 17:49:56, eae wrote:
> Ah, I assumed each assert would get a pass line (like they do for FAIL lines).
> 
> Whatever you prefer.

Thanks, I'll leave it as it is then.

[commit-bot: I haz the power, 2016-07-12 19:03:25]

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-07-12 19:46:16]

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

[commit-bot: I haz the power, 2016-07-12 19:46:17]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[cbiesinger, 2016-07-12 19:47:12]

The CQ bit was checked by cbiesinger@chromium.org

[commit-bot: I haz the power, 2016-07-12 19:48:59]

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-07-12 21:20:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-12 21:21:14]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-12 21:21:49]

Description was changed from

==========
Convert some flexbox tests to testharness.js

R=eae@chromium.org,dgrogan@chromium.org
==========

to

==========
Convert some flexbox tests to testharness.js

R=eae@chromium.org,dgrogan@chromium.org

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

[commit-bot: I haz the power, 2016-07-12 21:21:50]

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