1424122 - Crash in webrender::platform::windows::font::FontContext::create_glyph_analysis

Status: RESOLVED FIXED

(Core :: Graphics: WebRender, defect, P3)

Description

[Darkspirit]

Seen on Socorro.

bp-7a26273d-34c6-49a9-937d-5915e0171130 Build 20171129111030 (2017-11-29) @ Windows 7
> assertion failed: hr == 0

>  GraphicsCriticalError 	
> |[G0][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=448.172) |[G571][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=1073.89) |[G572][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=1073.89) |[G558][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.016) |[G559][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.066) |[G560][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.066) |[G561][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.119) |[G562][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.119) |[G563][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.119) |[G564][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.169) |[G565][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.169) |[G566][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.222) |[G567][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=471.275) |[G568][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=474.017) |[G569][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=474.33) |[G570][GFX1-]: WR: ERROR: Invalid window dimensions! Please call api.set_window_size() (t=1073.89) 

> 0 	xul.dll 	std::panicking::rust_panic_with_hook 	src/libstd/panicking.rs:583
> 1 	xul.dll 	std::panicking::begin_panic<str*> 	src/libstd/panicking.rs:538
> 2 	xul.dll 	webrender::platform::windows::font::FontContext::create_glyph_analysis 	gfx/webrender/src/platform/windows/font.rs:209
> 3 	xul.dll 	rayon::iter::internal::bridge_producer_consumer::helper<rayon::slice::IterProducer<webrender::glyph_rasterizer::GlyphRequest>, rayon::iter::map::MapConsumer<rayon::iter::collect::consumer::CollectConsumer<webrender::glyph_rasterizer::GlyphRasterJob>, closure>> 	third_party/rust/rayon/src/iter/internal.rs:268
> 4 	xul.dll 	rayon_core::job::{{impl}}::execute<closure> 	third_party/rust/rayon-core/src/job.rs:142
> 5 	xul.dll 	rayon_core::registry::in_worker<closure, ((), ())> 	third_party/rust/rayon-core/src/registry.rs:588
> 6 	xul.dll 	rayon::iter::internal::bridge_producer_consumer::helper<rayon::slice::IterProducer<webrender::glyph_rasterizer::GlyphRequest>, rayon::iter::map::MapConsumer<rayon::iter::collect::consumer::CollectConsumer<webrender::glyph_rasterizer::GlyphRasterJob>, closure>> 	third_party/rust/rayon/src/iter/internal.rs:275
> 7 	xul.dll 	rayon_core::registry::in_worker<closure, ((), ())> 	third_party/rust/rayon-core/src/registry.rs:588
> 8 	xul.dll 	rayon::iter::internal::bridge_producer_consumer::helper<rayon::slice::IterProducer<webrender::glyph_rasterizer::GlyphRequest>, rayon::iter::map::MapConsumer<rayon::iter::collect::consumer::CollectConsumer<webrender::glyph_rasterizer::GlyphRasterJob>, closure>> 	third_party/rust/rayon/src/iter/internal.rs:275
> 9 	xul.dll 	rayon_core::job::{{impl}}::execute<closure> 	third_party/rust/rayon-core/src/job.rs:142
> 10 	xul.dll 	rayon_core::registry::WorkerThread::wait_until<rayon_core::latch::CountLatch> 	third_party/rust/rayon-core/src/registry.rs:433
> 11 	xul.dll 	std::sys_common::backtrace::__rust_begin_short_backtrace<closure, ()> 	src/libstd/sys_common/backtrace.rs:131
> 12 	xul.dll 	alloc::boxed::{{impl}}::call_box<(), closure> 	src/liballoc/boxed.rs:725
> 13 	xul.dll 	std::sys::imp::thread::{{impl}}::new::thread_start 	src/libstd/sys/windows/thread.rs:54
> 14 		@0x900a2 	
> 15 	ntdll.dll 	RtlUserThreadStart 	
> 16 	kernel32.dll 	BasepReportFault 	
> 17 	kernel32.dll 	BasepReportFault

Comment 1

[Jeff Muizelaar [:jrmuizel]]

If this is only happening on Win7 it doesn't block release.

Comment 3

[Darkspirit]

Also happening on Win10:
https://crash-stats.mozilla.com/search/?app_notes=~WR%2B&build_id=%3E%3D20181212051241&proto_signature=~FontContext&platform=Windows&process_type=gpu&date=%3E%3D2018-12-10T19%3A01%3A42.000Z&date=%3C2018-12-17T19%3A01%3A42.000Z&_sort=-date&_facets=signature&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#crash-reports

Comment 4

[Kartikaya Gupta (email:[e-mail])]

Parking with Lee because fonts.