1176637 - 798853.html is going to permafail on Linux when Gecko 41 merges to Beta (Failed to create scaled font: 'cairo_scaled_font_status(scaledFont) == CAIRO_STATUS_SUCCESS')

Status: VERIFIED FIXED

(Core :: Graphics, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

John, I'm guessing this is related to bug 1176275?

https://treeherder.mozilla.org/logviewer.html#?job_id=8742458&repo=try

10:29:11 INFO - REFTEST TEST-START | file:///builds/slave/test/build/tests/reftest/tests/gfx/tests/crashtests/798853.html
10:29:11 INFO - REFTEST TEST-LOAD | file:///builds/slave/test/build/tests/reftest/tests/gfx/tests/crashtests/798853.html | 883 / 2836 (31%)
10:29:11 INFO - ++DOMWINDOW == 95 (0x7210e000) [pid = 1831] [serial = 2097] [outer = 0x9e661800]
10:29:11 INFO - --DOCSHELL 0x9b71b000 == 7 [pid = 1831] [id = 147]
10:29:11 INFO - [1831] ###!!! ASSERTION: Failed to create scaled font: 'cairo_scaled_font_status(scaledFont) == CAIRO_STATUS_SUCCESS', file /builds/slave/try-lx-d-000000000000000000000/build/src/gfx/thebes/gfxFontconfigFonts.cpp, line 2084
10:29:11 INFO - #01: gfxFcFont::GetOrMakeFont(_FcPattern*, _FcPattern*, gfxFontStyle const*) [gfx/thebes/gfxFontconfigFonts.cpp:2083]
10:29:11 INFO - #02: gfxFcFontSet::GetFontAt(unsigned int, gfxFontStyle const*) [gfx/thebes/gfxFontconfigFonts.cpp:725]
10:29:11 INFO - #03: gfxPangoFontGroup::GetBaseFont() [gfx/thebes/gfxTextRun.h:903]
10:29:11 INFO - #04: gfxPangoFontGroup::GetFirstValidFont(unsigned int) [gfx/thebes/gfxFontconfigFonts.cpp:1316]
10:29:11 INFO - #05: nsFontMetrics::GetMetrics(gfxFont::Orientation) const [gfx/thebes/gfxFont.h:1501]
10:29:11 INFO - #06: nsFontMetrics::GetMetrics() const [gfx/src/nsFontMetrics.h:239]
10:29:11 INFO - #07: nsFontMetrics::ExternalLeading() [gfx/src/nsFontMetrics.cpp:237]
10:29:11 INFO - #08: nsHTMLReflowState::CalcLineHeight(nsIContent*, nsStyleContext*, int, float) [layout/generic/nsHTMLReflowState.cpp:2570]
10:29:11 INFO - #09: nsHTMLReflowState::CalcLineHeight() const [layout/generic/nsHTMLReflowState.cpp:2639]
10:29:12 INFO - #10: nsBlockReflowState::nsBlockReflowState(nsHTMLReflowState const&, nsPresContext*, nsBlockFrame*, bool, bool, bool, int) [layout/generic/nsBlockReflowState.cpp:146]
10:29:12 INFO - #11: nsBlockFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsBlockFrame.cpp:1098]
10:29:12 INFO - #12: nsBlockReflowContext::ReflowBlock(mozilla::LogicalRect const&, bool, nsCollapsingMargin&, int, bool, nsLineBox*, nsHTMLReflowState&, unsigned int&, nsBlockReflowState&) [layout/generic/nsBlockReflowContext.cpp:298]
10:29:12 INFO - #13: nsBlockFrame::ReflowBlockFrame(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:3337]
10:29:12 INFO - #14: nsBlockFrame::ReflowLine(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:2710]
10:29:12 INFO - #15: nsBlockFrame::ReflowDirtyLines(nsBlockReflowState&) [layout/generic/nsColumnSetFrame.cpp:1152]
10:29:12 INFO - #16: nsBlockFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsBlockFrame.cpp:1173]
10:29:12 INFO - #17: nsBlockReflowContext::ReflowBlock(mozilla::LogicalRect const&, bool, nsCollapsingMargin&, int, bool, nsLineBox*, nsHTMLReflowState&, unsigned int&, nsBlockReflowState&) [layout/generic/nsBlockReflowContext.cpp:298]
10:29:12 INFO - #18: nsBlockFrame::ReflowBlockFrame(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:3337]
10:29:12 INFO - #19: nsBlockFrame::ReflowLine(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:2710]
10:29:12 INFO - #20: nsBlockFrame::ReflowDirtyLines(nsBlockReflowState&) [layout/generic/nsColumnSetFrame.cpp:1152]
10:29:12 INFO - #21: nsBlockFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsBlockFrame.cpp:1173]
10:29:12 INFO - #22: nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, mozilla::WritingMode const&, mozilla::LogicalPoint const&, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:981]
10:29:12 INFO - #23: nsCanvasFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsCanvasFrame.cpp:681]
10:29:12 INFO - #24: nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, mozilla::WritingMode const&, mozilla::LogicalPoint const&, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:981]
10:29:12 INFO - #25: nsHTMLScrollFrame::ReflowScrolledFrame(ScrollReflowState*, bool, bool, nsHTMLReflowMetrics*, bool) [layout/generic/nsGfxScrollFrame.cpp:524]
10:29:12 INFO - #26: nsHTMLScrollFrame::ReflowContents(ScrollReflowState*, nsHTMLReflowMetrics const&) [layout/generic/nsGfxScrollFrame.cpp:653]
10:29:12 INFO - #27: nsHTMLScrollFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsGfxScrollFrame.cpp:871]
10:29:12 INFO - #28: nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, int, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:1023]
10:29:12 INFO - #29: ViewportFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsViewportFrame.cpp:219]
10:29:12 INFO - #30: PresShell::DoReflow(nsIFrame*, bool) [layout/generic/nsHTMLReflowMetrics.h:273]
10:29:12 INFO - #31: PresShell::ProcessReflowCommands(bool) [layout/base/nsPresShell.cpp:9146]
10:29:12 INFO - #32: PresShell::FlushPendingNotifications(mozilla::ChangesToFlush) [layout/base/nsPresShell.cpp:4078]
10:29:12 INFO - #33: nsRefreshDriver::Tick(long long, mozilla::TimeStamp) [xpcom/base/nsRefPtr.h:231]
10:29:12 INFO - #34: mozilla::RefreshDriverTimer::Tick(long long, mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:186]
10:29:12 INFO - #35: mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::TickRefreshDriver(mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:437]
10:29:12 INFO - #36: nsRunnableMethodImpl<void (mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::*)(mozilla::TimeStamp), true, mozilla::TimeStamp>::Run() [xpcom/glue/nsThreadUtils.h:631]
10:29:12 INFO - #37: nsThread::ProcessNextEvent(bool, bool*) [xpcom/threads/nsThread.cpp:848]
10:29:12 INFO - #38: NS_ProcessNextEvent(nsIThread*, bool) [xpcom/glue/nsThreadUtils.cpp:265]
10:29:12 INFO - #39: mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) [ipc/glue/MessagePump.cpp:96]
10:29:12 INFO - #40: MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:234]
10:29:12 INFO - #41: MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:520]
10:29:12 INFO - #42: nsBaseAppShell::Run() [widget/nsBaseAppShell.cpp:167]
10:29:12 INFO - #43: nsAppStartup::Run() [toolkit/components/startup/nsAppStartup.cpp:281]
10:29:12 INFO - #44: XREMain::XRE_mainRun() [toolkit/xre/nsAppRunner.cpp:4264]
10:29:12 INFO - #45: XREMain::XRE_main(int, char**, nsXREAppData const*) [toolkit/xre/nsAppRunner.cpp:4347]
10:29:12 INFO - #46: XRE_main [toolkit/xre/nsAppRunner.cpp:4437]
10:29:12 INFO - #47: do_main [browser/app/nsBrowserApp.cpp:214]
10:29:12 INFO - #48: main [browser/app/nsBrowserApp.cpp:480]
10:29:12 INFO - REFTEST TEST-PASS | file:///builds/slave/test/build/tests/reftest/tests/gfx/tests/crashtests/798853.html | (LOAD ONLY)
10:29:12 INFO - REFTEST INFO | Loading a blank page
10:29:12 INFO - ++DOMWINDOW == 96 (0x6c196c00) [pid = 1831] [serial = 2098] [outer = 0x9e661800]
10:29:12 INFO - --DOMWINDOW == 95 (0x98fb2400) [pid = 1831] [serial = 1962] [outer = (nil)] [url = file:///builds/slave/test/build/tests/reftest/tests/gfx/tests/crashtests/395335-1.xhtml]
10:29:12 INFO - REFTEST TEST-UNEXPECTED-FAIL | file:///builds/slave/test/build/tests/reftest/tests/gfx/tests/crashtests/798853.html | assertion count 1 is more than expected 0 assertions
10:29:12 INFO - REFTEST TEST-END | file:///builds/slave/test/build/tests/reftest/tests/gfx/tests/crashtests/798853.html

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Any news here? We're less than a week away from the uplift.

Comment 2

[John Daggett (:jtd)]

Karl, could I ask you to take a quick look at this? This is a problem that was introduced after the fontconfig platform fontlist landed and was hidden because nightly/aurora has that branch enabled.

Comment 3

[John Daggett (:jtd)]

Attachment: patch, annotate the assertion

Annotate the assertion as it seems to be benign. There's already bug 868792 to fix this but since it's in the pango fontgroup code which may become obsolete I think we should defer fixing it for now.

Comment 4

[John Daggett (:jtd)]

(In reply to John Daggett (:jtd) from comment #2)
> Karl, could I ask you to take a quick look at this? This is a problem that
> was introduced after the fontconfig platform fontlist landed and was hidden
> because nightly/aurora has that branch enabled.

Er, never mind. Bug 868792 already has been logged and I'm not sure it's really important to fix at this point.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6c5f55b6ed6f

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6c5f55b6ed6f

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/3093906fb35e