Closed Bug 1389133 Opened 3 years ago Closed 3 years ago
_widget _get _style _context: assertion 'GTK _IS _WIDGET (widget)' failed
59 bytes, text/x-review-board-request
There's a minor regression from Bug 1365556. MOZ_GTK_COMBOBOX_SEPARATOR may not be defined for some themes so GetWidget(MOZ_GTK_COMBOBOX_SEPARATOR) can return null. We count with that: https://dxr.mozilla.org/mozilla-central/source/widget/gtk/gtk3drawing.cpp#1122 but the patch from Bug 1365556 always expect widget != null and calls gtk_widget_get_style_context(widget) on it which causes the gtk_widget_get_style_context: assertion 'GTK_IS_WIDGET (widget)' failed message.
Comment on attachment 8895886 [details] Bug 1389133 - don't invalidate style content for missing widgets, https://reviewboard.mozilla.org/r/167168/#review172578 Thank you :)
Attachment #8895886 - Flags: review?(karlt) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/dcbf487465de don't invalidate style content for missing widgets, r=karlt
Please nominate this for Beta approval when you get a chance.
Comment on attachment 8895886 [details] Bug 1389133 - don't invalidate style content for missing widgets, Approval Request Comment [Feature/Bug causing the regression]: Bug 1365556 [User impact if declined]: Error messages on console, crash on some systems (Bug 1388810) [Is this code covered by automated tests?]: no [Has the fix been verified in Nightly?]: yes [Needs manual test from QE? If yes, steps to reproduce]: no [List of other uplifts needed for the feature/fix]: none [Is the change risky?]: no [Why is the change risky/not risky?]: not risky, it's a straightforward null pointer check [String changes made/needed]: none
Attachment #8895886 - Flags: approval-mozilla-beta?
Comment on attachment 8895886 [details] Bug 1389133 - don't invalidate style content for missing widgets, Fix a regression. Beta56+.
Attachment #8895886 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.