943622 - Canvas strokeText hang

Status: RESOLVED WORKSFORME

(Core :: Graphics: Canvas2D, defect)

Description

[Jesse Ruderman]

Attachment: testcase

Maybe related to bug 899517?

Comment 1

[Jesse Ruderman]

Attachment: hang sample

Comment 2

[Alice0775 White]

There are 2 regression.
#1 hang for 10-20sec
  regressed by: bug 866747
#2 hang forever
  regressed by: bug 918613


#1 Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/b01fa7ae9238
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130430 Firefox/23.0 ID:20130430170720
Bad(hang for 10-20sec):
http://hg.mozilla.org/mozilla-central/rev/9f86b0aa679e
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130430 Firefox/23.0 ID:20130430185024
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=b01fa7ae9238&tochange=9f86b0aa679e

In local build,
Last Good: a0d7d78facd4
First Bad: 9bc277eb0698

#1 Regressed by:
9bc277eb0698	Jonathan Kew — bug 866747 - stroke glyphs individually instead of making a single path for the entire text run. r=bas



#2 Regression window(m-c)
Bad(hang for 10-20sec):
http://hg.mozilla.org/mozilla-central/rev/2316f49caa81
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:27.0) Gecko/20100101 Firefox/27.0 ID:20130929023534
Bad(hang forever):
http://hg.mozilla.org/mozilla-central/rev/8f805d3ef377
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:27.0) Gecko/20100101 Firefox/27.0 ID:20130929030202
#2 Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2316f49caa81&tochange=8f805d3ef377


#2 Regression window(m-i)
Bad(hang for 10-20sec):
http://hg.mozilla.org/integration/mozilla-inbound/rev/92f573a2f75d
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:27.0) Gecko/20100101 Firefox/27.0 ID:20130928052734
Bad(hang forever):
http://hg.mozilla.org/integration/mozilla-inbound/rev/3287d7bbdc14
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:27.0) Gecko/20100101 Firefox/27.0 ID:20130928072134
#2 Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=92f573a2f75d&tochange=3287d7bbdc14

#2 Regressed by:
3287d7bbdc14	Bas Schouten — Bug 918613: Convert cairo path code to use cairo_path_t. r=jrmuizel

Comment 3

[Guilherme Lima]

At least for me, on Win7 - this hangs only for a couple of seconds now.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

I was able to reproduce the hangs on Linux builds from around the time the bug was filed, but things are fine now. Alice, can you confirm that it's the same for you?

Comment 5

[Guilherme Lima]

I get 
(#0) 	CP+[GFX1-]: [D2D1.1] Attempt to use unsupported surface size Size(300,10000)
(#1) 	CP+[GFX1-]: Failed to create DrawTarget, Type: 5 Size: Size(300,10000)

Comment 6

[Alice0775 White]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #4)
> I was able to reproduce the hangs on Linux builds from around the time the
> bug was filed, but things are fine now. Alice, can you confirm that it's the
> same for you?

No longer reproduce the hang on Windows10.