The default bug view has changed. See this FAQ.

Custom CSS cursor not working

RESOLVED FIXED in mozilla1.9.3a2

Status

()

Core
Widget: Win32
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: bz, Assigned: robarnold)

Tracking

({regression})

Trunk
mozilla1.9.3a2
x86
Windows Vista
regression
Points:
---

Firefox Tracking Flags

(blocking1.9.2 .7+, status1.9.2 .7-fixed)

Details

Attachments

(2 attachments)

It looks like between 3.5 and 3.6 at least some uses of custom cursors broke.  Testcase is attached.  This is only a problem on Windows: on Mac they never worked and on Linux they seem to still be working.

Martijn, Ria, could one of you maybe hunt down the regression range for this?  Hovering over the little orange arrow should show a cursor that has a hand and the text "view gallery" in it.
status1.9.2: --- → ?
Keywords: regression
OS: Mac OS X → Windows Vista
Created attachment 422813 [details]
Testcase
I get this regression range:

works:
2009-07-28-04-mozilla-central
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2a1pre) Gecko/20090728 Minefield/3.6a1pre

broken:
2009-07-29-04-mozilla-central
Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2a1pre) Gecko/20090729 Minefield/3.6a1pre
Bug 502710 in that range looks pretty suspicious.  Rob, can you take a look at this, please?
Blocks: 502710
(Assignee)

Comment 4

7 years ago
Rather suspicious, but it should have been just code movement and a smidge of refactoring. I will probably get to this next week - I don't think it would be tricky to find out why but I've been wrong about that before :)
(Assignee)

Comment 5

7 years ago
Created attachment 427222 [details] [diff] [review]
Remove GetSystemMetric check
[Checkin: Comment 7]

It seems that I added a sanity check that the given cursor size should not exceed the system icon size. The system cursor size is 32x32 so even that is not big enough. There is a check for the size along the existing codepath for setting cursors however there is other code using this helper function that does not have similar checks (the tab preview code iirc). However, I think that will fail gracefully so it should be ok just to go ahead and remove the check from the helper function.
Assignee: nobody → tellrob
Status: NEW → ASSIGNED
Attachment #427222 - Flags: review?(jmathies)

Updated

7 years ago
Attachment #427222 - Flags: review?(jmathies) → review+
Duplicate of this bug: 543691
blocking2.0: --- → ?
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
Comment on attachment 427222 [details] [diff] [review]
Remove GetSystemMetric check
[Checkin: Comment 7]


http://hg.mozilla.org/mozilla-central/rev/f59d3c5a7ac7
Attachment #427222 - Attachment description: Remove GetSystemMetric check → Remove GetSystemMetric check [Checkin: Comment 7]
Attachment #427222 - Flags: approval1.9.2.2?
Status: ASSIGNED → RESOLVED
blocking2.0: ? → ---
Last Resolved: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a2
Comment on attachment 427222 [details] [diff] [review]
Remove GetSystemMetric check
[Checkin: Comment 7]

I guess there's no way to automate that testcase?
Attachment #427222 - Flags: approval1.9.2.2? → approval1.9.2.3?
(Assignee)

Updated

7 years ago
Duplicate of this bug: 555498

Comment 10

7 years ago
Broke again

While it worked indeed in a nightly build, it is broken again in 
rv:1.9.2.2 Gecko/20100316 Firefox/3.6.2
Just check the test case from the older comment.
Please check and resolve again! Thank you!
(Assignee)

Comment 11

7 years ago
(In reply to comment #10)
> Broke again
> 
> While it worked indeed in a nightly build, it is broken again in 
> rv:1.9.2.2 Gecko/20100316 Firefox/3.6.2
> Just check the test case from the older comment.
> Please check and resolve again! Thank you!

The patch is still waiting approval to land on the 3.6 branch. Once it gets approval, it can land and you'll see it in the next Firefox 3.6 release.

Comment 12

7 years ago
Any reason why this gets no 1.9.2 approval?

If this will not land in 1.9.2, let me know. I'll have to update the MDC docs then.
Requesting blocking for this web functionality regression.
blocking1.9.2: --- → ?

Updated

7 years ago
blocking1.9.2: ? → .5+
status1.9.2: ? → wanted

Comment 14

7 years ago
Comment on attachment 427222 [details] [diff] [review]
Remove GetSystemMetric check
[Checkin: Comment 7]

a=LegNeato for 1.9.2.5. Please ONLY land this on mozilla-1.9.2 default, as we
are still working on 1.9.2.4 on the relbranch
Attachment #427222 - Flags: approval1.9.2.4? → approval1.9.2.5+
blocking1.9.2: .5+ → .6+
Attachment #427222 - Flags: approval1.9.2.5+ → approval1.9.2.6+
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/d64dce8fb27c
status1.9.2: wanted → .6-fixed

Comment 16

7 years ago
1. "Platform: x86 Windows Vista", but also x86 Windows XP
2. "status1.9.2: .7-fixed" means, that version 1.9.2.7 will be fixed? That would be great, because (rv:1.9.2.6) Gecko/20100625 Firefox/3.6.6 is not fixed yet.
3. Allowing bigger cursors (again) increases the importance (well, if one would call it importance at all) of resolving 
Bug 248718  - Tooltip generated by title attribute appears unappropriately compared to current cursor.  
as well as
Bug 296191  - Custom CSS cursors [cursor: url(...);] sometimes overlay the link tooltips [title="..."] ...

Thanks for your great work!
(And if this bug will be fixed in 1.9.2.7: Thanks for fixing!)
> 2. "status1.9.2: .7-fixed" means, that version 1.9.2.7 will be fixed?

Yes, unless there's another unscheduled firedrill release.
You need to log in before you can comment on or make changes to this bug.