Closed
Bug 1163993
Opened 10 years ago
Closed 10 years ago
[gtk3] crash @ nsNativeThemeGTK::DrawWidgetBackground()
Categories
(Core :: Widget: Gtk, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 1127752
| Tracking | Status | |
|---|---|---|
| firefox41 | --- | affected |
People
(Reporter: stransky, Unassigned)
References
Details
Attachments
(1 file)
|
908 bytes,
patch
|
Details | Diff | Splinter Review |
Reproduction steps:
go to duckduckgo.com, enter some text to search box
in nsNativeThemeGTK::DrawWidgetBackground()
900 cairo_t *cairo_ctx =
901 (cairo_t*)aDrawTarget.GetNativeSurface(NativeSurfaceType::CAIRO_CONTEXT);
cairo_ctx is null.
|
Reporter | |
Comment 1•10 years ago
|
||
looks like the aDrawTarget is mozilla::gfx::DrawTargetDual type which does not have cairo context.
|
|
Reporter | |
Comment 2•10 years ago
|
||
I'm not sure this is the correct solution but it prevents the browser from crashes at least.
|
|
Reporter | |
Comment 3•10 years ago
|
||
Comment on attachment 8605210 [details] [diff] [review]
workaround
Karl, what do you think? Do we need to support the dual target?
Attachment #8605210 -
Flags: feedback?(karlt)
|
||
Comment 4•10 years ago
|
||
Comment on attachment 8605210 [details] [diff] [review]
workaround
Do the changes from bug 1127752 help here?
If not, its probably best to ask Lee what the best solution here would be.
(He will be more likely to know about DrawTargetDual than I.)
Attachment #8605210 -
Flags: feedback?(karlt)
|
|
Reporter | |
Comment 5•10 years ago
|
||
Bug 1127752 seems to fixed that, Thanks.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•