Bug 122853 - Drop CSS cursor quirk for missing fallback cursor
Summary: Drop CSS cursor quirk for missing fallback cursor
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BlinkMergeCandidate
Depends on:
Blocks:
 
Reported: 2013-10-15 12:38 PDT by Ryosuke Niwa
Modified: 2022-08-06 21:00 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2013-10-15 12:38:23 PDT
Consider merging https://chromium.googlesource.com/chromium/blink/+/db44f7cb06a22c64e63b2dcb35a7a59b858f6c33

IE used to allow URI cursors without a specified pre-defined cursor value
fallback. This CL removes the support for that quirk because IE allegedly
dropped this quirk in IE9, even in the compat modes (see issue 286668).

Made the existing parsing tests run in strict mode and moved test cases for
quirks mode to a new file.

The 'hand' -> 'pointer' quirk was not touched.
Comment 1 Ahmad Saleem 2022-08-06 19:42:07 PDT
I changed the test case of quirk added in the patch to following JSFiddle:

Link - https://jsfiddle.net/npusjzhm/1/show

It showed following across all browsers:

Test cursor rules which should accept 'hand' as an alias of 'pointer'.
FAIL roundtripCssRule("cursor: hand;") should be cursor: pointer;. Was .
FAIL roundtripCssRule("cursor: url(file:///foo.png), hand;") should be cursor: url(file:///foo.png), pointer;. Was .

Test invalid cursor rules which shouldn't parse at all.
PASS roundtripCssRule("cursor: url(file:///foo.png), url(file:///foo2.png);") is ""
PASS successfullyParsed is true

___

rniwa@webkit.org - is it needed now? or this is resolved? Thanks!
Comment 2 Ryosuke Niwa 2022-08-06 21:00:38 PDT
We match Blink in this regard now.