207301 - After clicking links or buttons cursor becomes a zoom-in magnifying glass

Status: VERIFIED FIXED

(Camino Graveyard :: General, defect)

Description

[Xavier Snelgrove]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4b) Gecko/20030527 Camino/0.7+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4b) Gecko/20030527 Camino/0.7+

If you click on any links in a webpage or on menubar links the cursor becomes a
zoom-in magnifying glass. This dissapears after some normal use.

Reproducible: Always

Steps to Reproduce:
1.Click "home" button on menubar
or
1.Click any hyperlink

Actual Results:  
the cursor became a zoom-in magnifying glass

Expected Results:  
the cursor should have stayed the same.

Comment 1

[William K.]

Ive noticed this too starting with 2003052704

Comment 2

[Chris Petersen]

Confirmed. This regression starts on the 2003-05-27-04 NB. The application uses
the magnifying glass as a wait cursor. This magnifying glass cursor tends to
appear while loading page content or large images.

Comment 3

[Derek Schrock]

Confirmed with 2003052704

Comment 4

[Jasper]

Confirmed with 2003052804

Comment 5

[Mike Pinkerton (not reading bugmail)]

who landed this feature on the trunk? does mozilla have the same issue?

Comment 6

[Chris Petersen]

This doesn't happen in the Mozilla Macho trunk (2003-05-28-08) just in the
Camino NB. This was working fine on the 05/26 build but regressed on the 05/27
build. Perhaps this is the culprit ? 

http://bugzilla.mozilla.org/show_bug.cgi?id=78363

This checkin was done on 5.26 ..

Comment 7

[Chris Petersen]

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

Comment 8

[Jan Varga [:janv]]

Attachment: fix

yeah, this is a regression from bug 78363

Comment 9

[Jan Varga [:janv]]

reassigning

Comment 10

[Andrew Thompson]

Yes. This was my fault. Essentially a 2 part problem:

First, fix I was working on that went into the trunk that didn't go into Camino
because discussion with smfr/pink suggested a different approach for Camino
based on Cocoa's NSCursor class. This is something I'm working on that I didn't
finish yet

Second, I fixed a regression caused by an interaction between a bug varga fixed
and a bug bryner was working on and some related bitrot with one of my patches.
I decided to change the absolute numerical ids of some cursors to avoid a
recurrance of the problem, but I only fixed it in Mozilla, not Camino :(

I'm afraid the patch as given won't work. I'll attach something momentarily...

Comment 11

[Andrew Thompson]

Attachment: Fix also reassigns wait cursor. Temporary fix until camino cursor reimplementation done

See previous comment. This will ensure cursor ids are correct in Camino until
previously discussed Cocoa NSCursor implementation is completed.

Comment 12

[Andrew Thompson]

Comment on attachment 124416 [details] [diff] [review]
Fix also reassigns wait cursor. Temporary fix until camino cursor reimplementation done

varga: thank for trying to fix my mistake.

pink/smfr: sorry for screwing up Camino. I didn't see this in my tree due to a
partial fix I have locally but didn't check in due to discussion of a different
way of doing Camino cursors based on NSCursor. Mea culpa.

Comment 13

[Jan Varga [:janv]]

no problem

Comment 14

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 124416 [details] [diff] [review]
Fix also reassigns wait cursor. Temporary fix until camino cursor reimplementation done

r=pink

Comment 15

[Andrew Thompson]

I have to go to work. Will check in this evening, unless someone else wants to
do it in the next few hours.

Comment 16

[Mike Pinkerton (not reading bugmail)]

landed on the trunk, should we try to get this on the branch?

Comment 17

[Greg K.]

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

Comment 18

[Andrew Thompson]

Doesn't need to go on the branch, because I didn't put the fix for bug 78363
(cause of this regression) onto the branch.

Comment 19

[Chris Petersen]

Verified on the camino 2003-05-30-04 NB.