Bug 11752 – [CSS 3] missing support for cursor selector 'copy'

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Lars Næsbye Christensen]

The CSS cursor selector is not supported by WebKit.

[Lars Næsbye Christensen]

Created attachment 11734 [details]
patch to support CSS 3 cursor selector 'copy'

The next-to-last we need for full cursor selector support....'none' is a bit trickier.....

[Alexey Proskuryakov]

Comment on attachment 11734 [details]
patch to support CSS 3 cursor selector 'copy'

r=me

[Alexey Proskuryakov]

Committed revision 18017.

[David Kilzer (:ddkilzer)]

Comment on attachment 11734 [details]
patch to support CSS 3 cursor selector 'copy'

>Index: WebCore/css/CSSValueKeywords.in
>===================================================================
>--- WebCore/css/CSSValueKeywords.in	(revision 18014)
>+++ WebCore/css/CSSValueKeywords.in	(working copy)
>@@ -170,7 +170,7 @@ no-repeat
> # CSS_PROP__WEBKIT_BACKGROUND_COMPOSITE:
> #
> clear
>-copy
>+# copy
> source-over
> source-in
> source-out

Did we really want to check that in?  Shouldn't that line just be deleted?

[David Kilzer (:ddkilzer)]

(In reply to comment #4)
> (From update of attachment 11734 [details] [edit])
> >Index: WebCore/css/CSSValueKeywords.in
> >===================================================================
> >--- WebCore/css/CSSValueKeywords.in	(revision 18014)
> >+++ WebCore/css/CSSValueKeywords.in	(working copy)
> >@@ -170,7 +170,7 @@ no-repeat
> > # CSS_PROP__WEBKIT_BACKGROUND_COMPOSITE:
> > #
> > clear
> >-copy
> >+# copy
> > source-over
> > source-in
> > source-out
> 
> Did we really want to check that in?  Shouldn't that line just be deleted?

Actually, I'm not sure that commenting that out is even correct?!

[Alexey Proskuryakov]

Yes - that's how duplicate names are treated elsewhere in this file.