1884566 - Firefox CSD show the wrong cursor theme

Status: ASSIGNED

(Core :: Widget: Gtk, defect, P2)

Description

[daf]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:123.0) Gecko/20100101 Firefox/123.0

Steps to reproduce:

When hovering the CSD certain parts will show wrong cursor theme.

KDE bug tracker report: https://bugs.kde.org/show_bug.cgi?id=482587

Actual results:

https://youtu.be/vT_CYkm7C-0

Expected results:

The correct cursor theme should be used

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Firefox::Theme' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Martin Stránský [:stransky] (ni? me)]

Is that a recent regression? Can you try mozregression tool to find broken commit?
Thanks.

Comment 3

[Martin Stránský [:stransky] (ni? me)]

There's a link:
https://fedoraproject.org/wiki/How_to_debug_Firefox_problems#Use_Mozregression_tool

Comment 4

[daf]

(In reply to Martin Stránský [:stransky] (ni? me) from comment #3)

There's a link:
https://fedoraproject.org/wiki/How_to_debug_Firefox_problems#Use_Mozregression_tool

5:18.15 INFO: Last good revision: 57b850c11aa0eb7411f592923df6906827f96786
5:18.15 INFO: First bad revision: c71d09ab8b2b32c1cc00a0eba6f8f3b72a61e5fa
5:18.15 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=57b850c11aa0eb7411f592923df6906827f96786&tochange=c71d09ab8b2b32c1cc00a0eba6f8f3b72a61e5fa

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

:jordan, since you are the author of the regressor, bug 1871863, could you take a look?

For more information, please visit BugBot documentation.

Comment 6

[Martin Stránský [:stransky] (ni? me)]

I'll look at it.

Comment 7

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1884566 [Linux] Allow to use legacy cursor if widget.gtk.legacy-cursors.enabled is set r?emilio

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1871863

Comment 9

[Dianna Smith [:diannaS]]

is there a severity for this?

Comment 10

[Emilio Cobos Álvarez [:emilio]]

So, the "right, then wrong" cursor shown in comment 0 is because of this, but it's unclear if the right cursor is ours or GTK's?

Comment 11

[Emilio Cobos Álvarez [:emilio]]

I see inconsistent cursors with such setup even on plasma's settings titlebar fwiw. So the issue isn't specific to Firefox either.

Comment 12

[Martin Stránský [:stransky] (ni? me)]

(In reply to Emilio Cobos Álvarez (:emilio) from comment #10)

So, the "right, then wrong" cursor shown in comment 0 is because of this, but it's unclear if the right cursor is ours or GTK's?

The right cursor comes from Gtk and wrong from Firefox. We change it on motion-notify-event mContainer callback.

Comment 13

[Martin Stránský [:stransky] (ni? me)]

Perhaps better to move the patch to Bug 1876366.

Comment 14

[Phabricator Automation]

Comment on attachment 9390736 [details]
Bug 1884566 [Linux] Allow to use legacy cursor if widget.gtk.legacy-cursors.enabled is set r?emilio

Revision D204351 was moved to bug 1876366. Setting attachment 9390736 [details] to obsolete.

Comment 15

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1871863

Comment 16

[g.s]

The default for widget.gtk.legacy-cursors.enabled seems to have changed to false in 142, reintroducing thius bug.