Closed
Bug 723110
Opened 13 years ago
Closed 13 years ago
crash _moz_cairo_scaled_font_destroy with gfx.canvas.azure.enabled=true on Linux
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla13
People
(Reporter: m_kato, Assigned: gw280)
References
Details
(Keywords: crash)
Crash Data
Attachments
(2 files)
(deleted),
patch
|
jrmuizel
:
review-
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jrmuizel
:
review+
|
Details | Diff | Splinter Review |
When browse http://www.scirra.com/labs/perftest-2d/ with gfx.canvas.azure.enable=true, firefox for Linux crashes the following stack.
This bug was filed from the Socorro interface and is
report bp-dad23233-c988-4cad-aa7c-eaaf12120201 .
=============================================================
0 libxul.so _moz_cairo_scaled_font_destroy gfx/cairo/cairo/src/cairo-scaled-font.c:1220
1 libxul.so mozilla::gfx::ScaledFontBase::~ScaledFontBase gfx/2d/ScaledFontBase.cpp:92
2 libxul.so mozilla::gfx::ScaledFontBase::~ScaledFontBase gfx/2d/ScaledFontBase.cpp:94
3 libxul.so nsCanvasBidiProcessorAzure::DrawText content/canvas/src/nsCanvasRenderingContext2DAzure.cpp:3034
4 libxul.so nsBidiPresUtils::ProcessText layout/base/nsBidiPresUtils.cpp:1824
5 libxul.so nsCanvasRenderingContext2DAzure::DrawOrMeasureText content/canvas/src/nsCanvasRenderingContext2DAzure.cpp:3332
6 libxul.so nsIDOMCanvasRenderingContext2D_FillText obj-firefox/js/xpconnect/src/dom_quickstubs.cpp:2535
7 libxul.so js::InvokeKernel js/src/jscntxtinlines.h:311
8 libxul.so js::Interpret js/src/jsinterp.cpp:2801
9 libxul.so js::InvokeKernel js/src/jsinterp.cpp:537
10 libxul.so js::Invoke js/src/jsinterp.h:157
11 libxul.so JS_CallFunctionValue js/src/jsapi.cpp:5452
12 libxul.so nsJSContext::CallEventHandler dom/base/nsJSEnvironment.cpp:1973
13 libxul.so nsGlobalWindow::RunTimeout dom/base/nsGlobalWindow.cpp:9375
14 libxul.so nsGlobalWindow::TimerCallback dom/base/nsGlobalWindow.cpp:9763
15 libxul.so nsTimerImpl::Fire xpcom/threads/nsTimerImpl.cpp:428
16 libxul.so nsTimerEvent::Run xpcom/threads/nsTimerImpl.cpp:524
17 libxul.so nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:657
18 libxul.so NS_ProcessNextEvent_P obj-firefox/xpcom/build/nsThreadUtils.cpp:245
19 libxul.so mozilla::ipc::MessagePump::Run ipc/glue/MessagePump.cpp:134
20 libxul.so MessageLoop::Run ipc/chromium/src/base/message_loop.cc:208
21 libxul.so nsBaseAppShell::Run widget/xpwidgets/nsBaseAppShell.cpp:189
22 libxul.so nsAppStartup::Run toolkit/components/startup/nsAppStartup.cpp:220
23 libxul.so XRE_main toolkit/xre/nsAppRunner.cpp:3537
24 firefox main browser/app/nsBrowserApp.cpp:205
25 libc-2.13.so libc-2.13.so@0x2130c
26 firefox firefox@0x1b9f
Reporter | ||
Updated•13 years ago
|
Component: General → Graphics
Product: Firefox → Core
QA Contact: general → thebes
Hardware: All → x86_64
Updated•13 years ago
|
Assignee | ||
Comment 2•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #593549 -
Flags: review?(jmuizelaar)
Comment 3•13 years ago
|
||
Comment on attachment 593549 [details] [diff] [review]
Make sure mScaledFont is initialised if USE_CAIRO is on
Please implement ScaledFontFreetype and get rid of this constructor instead.
Attachment #593549 -
Flags: review?(jmuizelaar) → review-
Assignee | ||
Comment 4•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #593588 -
Flags: review?(jmuizelaar)
Updated•13 years ago
|
Attachment #593588 -
Flags: review?(jmuizelaar) → review+
Comment 5•13 years ago
|
||
Can we land this? With this bug resolved, we can test better Skia and if the performance is better than Cairo (and I think it is) we can switch to Skia by default.
Assignee | ||
Updated•13 years ago
|
Whiteboard: [autoland-try: 593588]
Assignee | ||
Updated•13 years ago
|
Whiteboard: [autoland-try: 593588] → [autoland-try:593588]
Updated•13 years ago
|
Whiteboard: [autoland-try:593588] → [autoland-in-queue]
Comment 6•13 years ago
|
||
Autoland Patchset:
Patches: 593588
Branch: mozilla-central => try
Destination: http://hg.mozilla.org/try/pushloghtml?changeset=a627a8131be2
Try run started, revision a627a8131be2. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=a627a8131be2
Comment 7•13 years ago
|
||
Try run for a627a8131be2 is complete.
Detailed breakdown of the results available here:
https://tbpl.mozilla.org/?tree=Try&rev=a627a8131be2
Results (out of 209 total builds):
success: 174
warnings: 21
failure: 14
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/autolanduser@mozilla.com-a627a8131be2
Updated•13 years ago
|
Whiteboard: [autoland-in-queue]
Updated•13 years ago
|
Target Milestone: --- → mozilla13
Comment 8•13 years ago
|
||
Comment 9•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•