242378 - Dynamically modifying cursor fails

Status: RESOLVED FIXED

(Core Graveyard :: GFX: Win32, defect)

Description

[Gérard Talbot]

The cursors "-moz-zoom-in" and "-moz-zoom-out" are no longer rendered at the URL
page; they were not too long ago (Mozilla 1.7; don't remember which version).
FWIW, Mozilla 1.4 and Mozilla 1.5 render accordingly the  "-moz-zoom-in" and
"-moz-zoom-out" cursors at the provided URL. I strongly suspect that this is a
regression which happened no more than 10 days ago.
I searched to find a duplicate for this and couldn't find any; possibly bug
195045 or bug 50511 could be but I din't think so.
Sorry if "Style System" is not the correct component (not sure exactly which
component, to be honest) or if there is a duplicate bug for this.

Mozilla 1.8 alpha build 2004050108 under XP Pro here.

Comment 1

[R.K.Aa.]

*** Bug 242233 has been marked as a duplicate of this bug. ***

Comment 2

[R.K.Aa.]

confirming based on dup and comment there

Comment 3

[Boris Zbarsky [:bzbarsky]]

Worksforme in Linux trunk build 2004-05-02-06.  See bug 241316, though.  That
one's Windows-only too...

> I strongly suspect that this is a regression which happened no more than 10
> days ago.

We have builds back to then on the ftp site, so if it's that recent a regression
you should be able to locate the exact day it happened.  When was that?

Comment 4

[Gérard Talbot]

Yes, bug 242233 and this bug refer to the same pair of cursors; I too don't get
the image resizing cursor under XP Pro.

I'm short on time these days so I won't be able to determine fast/soon the date
of build where the regression appeared. FWIW, bug 242233 was opened 4 days ago. 
All I can say for sure for now is that the provided URL used to render
dynamically the -moz-zoom-in and -moz-zoom-out cursors on a click and drag in
the image in recent Mozilla 1.7.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Well, since it works fine on Linux it's not style system, and since it works
fine on Linux I can't debug it, so....

Comment 6

[Warner Young]

WFM with Firefox 20040502 build, WinXP. Windows Mozilla only?

Comment 7

[Isaac Hwak Han]

Attachment: fix

Automatic license update mysteriously removed #include...

Comment 8

[Frank Wein [:mcsmurf]]

just for the record: This regressed before 20040424, but now that there is a
patch, there is no need for more testing :).

Comment 9

[Frank Wein [:mcsmurf]]

Comment on attachment 147529 [details] [diff] [review]
fix

Gerv can you review this (since it was caused by your relicensing change)?

Comment 10

[Gervase Markham [:gerv]]

Comment on attachment 147529 [details] [diff] [review]
fix

Hmm. That is mysterious. I'll point this out to the script author - although it
could have been a manual mistake as well.

r=gerv, along with my apologies; but you need to get a proper r= from the
module owner as well.

Gerv

Comment 11

[Frank Wein [:mcsmurf]]

Comment on attachment 147529 [details] [diff] [review]
fix

very simple review for you, only a one-liner, regression fix

Comment 12

[Christian :Biesinger (don't email me, ping me on IRC)]

checked in on trunk

Checking in widget/src/build/widget.rc;
/cvsroot/mozilla/widget/src/build/widget.rc,v  <--  widget.rc
new revision: 1.14; previous revision: 1.13
done