Rewrote :empty tests to avoid list-item bug.

Rewrote :empty tests to avoid list-item bug.

Run :empty tests as js-tests instead of layout tree dumps. The issue
causing the original test to be flaky is reported as crbug.com/613957.

R=bugsnash@chromium.org
BUG=610180
Committed: https://crrev.com/00bc83a8ed9ee64cd4b478cf760fa58809b95984
Cr-Commit-Position: refs/heads/master@{#395582}

[rune, 2016-05-23 09:58:21]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-05-23 09:58:31]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2006633002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2006633002/1

[rune, 2016-05-23 09:58:45]

ptal

[commit-bot: I haz the power, 2016-05-23 12:26:30]

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

[commit-bot: I haz the power, 2016-05-23 12:26:31]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rune, 2016-05-24 09:53:24]

rune@opera.com changed reviewers:
+ mstensho@opera.com

[mstensho (USE GERRIT), 2016-05-24 10:37:54]

lgtm with nit (fine to ignore, though)

https://codereview.chromium.org/2006633002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/fast/css/empty-pseudo-class.html (right):

https://codereview.chromium.org/2006633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/css/empty-pseudo-class.html:4: .empty:empty,
.not-empty { color: green }
For the sake of consistency, it would be nice if either both or none of the
selectors on this line had a pseudo class.

[rune, 2016-05-24 11:05:49]

The CQ bit was checked by rune@opera.com

[rune, 2016-05-24 11:05:50]

The patchset sent to the CQ was uploaded after l-g-t-m from mstensho@opera.com
Link to the patchset: https://codereview.chromium.org/2006633002/#ps20001
(title: "Better selectors in test")

[commit-bot: I haz the power, 2016-05-24 11:06:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2006633002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2006633002/20001

[commit-bot: I haz the power, 2016-05-24 13:03:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-24 13:04:46]

Description was changed from

==========
Rewrote :empty tests to avoid list-item bug.

Run :empty tests as js-tests instead of layout tree dumps. The issue
causing the original test to be flaky is reported as crbug.com/613957.

R=bugsnash@chromium.org
BUG=610180
==========

to

==========
Rewrote :empty tests to avoid list-item bug.

Run :empty tests as js-tests instead of layout tree dumps. The issue
causing the original test to be flaky is reported as crbug.com/613957.

R=bugsnash@chromium.org
BUG=610180

Committed: https://crrev.com/00bc83a8ed9ee64cd4b478cf760fa58809b95984
Cr-Commit-Position: refs/heads/master@{#395582}
==========

[commit-bot: I haz the power, 2016-05-24 13:04:48]

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