Closed Bug 877409 Opened 6 years ago Closed 6 years ago

Clean up obsolete GTK code

Categories

(Core :: Widget: Gtk, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla24

People

(Reporter: t_arceri, Assigned: t_arceri)

Details

Attachments

(2 files)

This bug is for two things:

1. To remove old GTK_CHECK_VERSION/gtk_check_version for versions of GTK already covered by the minimum GTK build requirement 2.10.

2. To remove depreciated GDK_ROOT_PARENT()
Patch to remove old GTK_CHECK_VERSION/gtk_check_version for versions of GTK already covered by the minimum GTK build requirement 2.10.
Attachment #755616 - Flags: review?(karlt)
Attachment #755616 - Flags: review?(karlt) → review+
Attachment #755616 - Flags: checkin?
Keywords: checkin-needed
Ok it turns out the remaining GDK_ROOT_PARENT() calls are inside a #if 0 in ./widget/gtk2/nsWindow.cpp

I have provided a patch to update this anyway. But should this code just be removed or is it still useful in some way?
Attachment #755807 - Flags: review?(karlt)
Attachment #755616 - Flags: checkin? → checkin+
Comment on attachment 755807 [details] [diff] [review]
Replace old calls

(In reply to Timothy Arceri from comment #2)
> But should this code just be removed or is it still useful in some way?

I don't mind.  Looks like it is debugging code that might still work, but I haven't used it.  It won't work with GTK3.  If you'd like to remove the paint flashing code, I'd review it.
Attachment #755807 - Flags: review?(karlt) → review+
Comment on attachment 755807 [details] [diff] [review]
Replace old calls

I'll just leave it alone for now.
Attachment #755807 - Flags: checkin?
Whiteboard: [leave open] → check
Keywords: checkin-needed
Whiteboard: check
Attachment #755807 - Flags: checkin? → checkin+
https://hg.mozilla.org/mozilla-central/rev/f73e297ed889
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.