Remove outdated list style types

Remove outdated list style types

Blink supports the list of list style types from the old draft:
http://www.w3.org/TR/2002/WD-css3-lists-20021107/

However a lot of these are removed in the latest version:
http://dev.w3.org/csswg/css-counter-styles-3/

So remove those types that are no longer in the latest spec. I verified
for these types that Firefox does not support them (anymore) either.

Note that the following styles are kept for compatibility with Firefox
which prefixes them with -moz:
hangul
hangul-consonant
ethiopic-halehame
ethiopic-halehame-am
ethiopic-halehame-ti-er
ethiopic-halehame-ti-et
urdu

Adopt tests to test only the list style types mentioned in the latest spec.

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

[rwlbuis, 2015-05-29 01:57:20]

rob.buis@samsung.com changed reviewers:
+ timloh@chromium.org

[rwlbuis, 2015-05-29 01:57:21]

PTAL.

This removes the list styles that I think are safe to remove based on FF:
https://developer.mozilla.org/en-US/docs/Web/CSS/list-style-type

I kept hangul* since FF supports it through -moz-hangul*.

Same for ethiopic-halehame-*, though it seems we do not have ethiopic-halehame
itself.

Finally I am undecided about the 4 failing tests ; I am not sure if we simply
need to remove the tests for the removed list styles or rewrite the tests to
expect default numeric/alphabetic list behavior.

[leviw_travelin_and_unemployed, 2015-05-29 18:08:44]

leviw@chromium.org changed reviewers:
+ leviw@chromium.org

[leviw_travelin_and_unemployed, 2015-05-29 18:08:45]

Seems like this is PSA-on-Blink-Dev-worthy, as it's visible to the web.

[rwlbuis, 2015-06-02 17:51:08]

Patchset #4 (id:60001) has been deleted

[rwlbuis, 2015-06-03 02:58:07]

On 2015/05/29 18:08:45, leviw wrote:
> Seems like this is PSA-on-Blink-Dev-worthy, as it's visible to the web.

Yes, that is a good idea, I can do that.

@timloh PTAL. I updated the tests to make more sense given the latest spec, also
updating references to the 7 feb CR.
I also added negative tests for the removed styles
(fast/lists/w3-css3-list-styles-deprecated.html), not sure if we need it.
The failing linux_blink_rel is unrelated to my change I think.

[Timothy Loh, 2015-06-04 04:10:55]

I think this is fine with a PSA to blink-dev :)

https://codereview.chromium.org/1152763006/diff/100001/Source/core/css/CSSPri...
File Source/core/css/CSSPrimitiveValueMappings.h (right):

https://codereview.chromium.org/1152763006/diff/100001/Source/core/css/CSSPri...
Source/core/css/CSSPrimitiveValueMappings.h:1573: case Hebrew:
You accidentally unsorted this item :)

[rwlbuis, 2015-06-04 21:40:31]

On 2015/06/04 04:10:55, Timothy Loh wrote:
> I think this is fine with a PSA to blink-dev :)

So that would be sent *after* the CL goes in?

>
https://codereview.chromium.org/1152763006/diff/100001/Source/core/css/CSSPri...
> File Source/core/css/CSSPrimitiveValueMappings.h (right):
> 
>
https://codereview.chromium.org/1152763006/diff/100001/Source/core/css/CSSPri...
> Source/core/css/CSSPrimitiveValueMappings.h:1573: case Hebrew:
> You accidentally unsorted this item :)

Ouch! Fixed :)

[leviw_travelin_and_unemployed, 2015-06-04 21:42:14]

On 2015/06/04 at 21:40:31, rob.buis wrote:
> On 2015/06/04 04:10:55, Timothy Loh wrote:
> > I think this is fine with a PSA to blink-dev :)
> 
> So that would be sent *after* the CL goes in?

It should come before.

> 
> >
https://codereview.chromium.org/1152763006/diff/100001/Source/core/css/CSSPri...
> > File Source/core/css/CSSPrimitiveValueMappings.h (right):
> > 
> >
https://codereview.chromium.org/1152763006/diff/100001/Source/core/css/CSSPri...
> > Source/core/css/CSSPrimitiveValueMappings.h:1573: case Hebrew:
> > You accidentally unsorted this item :)
> 
> Ouch! Fixed :)

[rwlbuis, 2015-06-04 23:24:19]

On 2015/06/04 21:42:14, leviw wrote:
> On 2015/06/04 at 21:40:31, rob.buis wrote:
> > On 2015/06/04 04:10:55, Timothy Loh wrote:
> > > I think this is fine with a PSA to blink-dev :)
> > 
> > So that would be sent *after* the CL goes in?
> 
> It should come before.

Thanks! I posted here:

https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/C_GpBXGLyOM

[Timothy Loh, 2015-06-09 01:00:56]

On 2015/06/04 23:24:19, rwlbuis wrote:
> On 2015/06/04 21:42:14, leviw wrote:
> > On 2015/06/04 at 21:40:31, rob.buis wrote:
> > > On 2015/06/04 04:10:55, Timothy Loh wrote:
> > > > I think this is fine with a PSA to blink-dev :)
> > > 
> > > So that would be sent *after* the CL goes in?
> > 
> > It should come before.
> 
> Thanks! I posted here:
> 
> https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/C_GpBXGLyOM

lgtm

[rwlbuis, 2015-06-09 10:20:37]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2015-06-09 10:20:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1152763006/120001

[commit-bot: I haz the power, 2015-06-09 11:38:40]

Committed patchset #6 (id:120001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196756