758980 - Certain font disappears when page scroll down and resize width

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Alice0775 White]

Attachment: html and font

Build Identifier:
http://hg.mozilla.org/mozilla-central/rev/133aa3a2ef0a
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/15.0 Firefox/15.0a1 ID:20120527030515

Font disappears when page scroll down and resize width

Steps to reproduce:

1. Install font JSGOTHIC.TTC
2. Open index.html
3. Scroll down and resize window width if necessary

Actual results:
  Font disappears

Expected results:
  Should not


Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/36e938e51481
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/15.0 Firefox/15.0a1 ID:20120523030525
Bad:
http://hg.mozilla.org/mozilla-central/rev/aa2b52bd0374
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=36e938e51481&tochange=aa2b52bd0374

Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/bbb12d0bcf49
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/15.0 Firefox/15.0a1 ID:20120521220651
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/abbf642df562
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/15.0 Firefox/15.0a1 ID:20120521222950
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=bbb12d0bcf49&tochange=abbf642df562

Triggered by:
abbf642df562	Bas Schouten — Bug 715768: Enable Azure-Thebes wrapper by default for D2D. r=jrmuizel


Setting gfx.content.azure.enabled to false helps

Comment 1

[Alice0775 White]

Graphics
Adapter Description : ATI Radeon HD 4300/4500 Series
Vendor ID : 0x1002
Device ID : 0x954f
Adapter RAM : 512
Adapter Drivers : aticfx64 aticfx64 aticfx32 aticfx32 atiumd64 atidxx64 atiumdag atidxx32 atiumdva atiumd6a atitmm64
Driver Version : 8.961.0.0
Driver Date : 4-5-2012
Direct2D Enabled : true
DirectWrite Enabled : true (6.1.7601.17789)
ClearType Parameters : Gamma: 2200 Pixel Structure: RGB ClearType Level: 50 Enhanced Contrast: 200 
WebGL Renderer : Google Inc. -- ANGLE (ATI Radeon HD 4300/4500 Series) -- OpenGL ES 2.0 (ANGLE 1.0.0.1041)
GPU Accelerated Windows : 1/1 Direct3D 10
AzureBackend : direct2d

Comment 2

[Alice0775 White]

Attachment: another html

Comment 3

[Alice0775 White]

Attachment: html

Comment 4

[Alice0775 White]

Attachment: another html

Comment 5

[Alice0775 White]

Screen capture: http://youtu.be/BQ8eW6YQrF8

Comment 6

[Alice0775 White]

And also setting gfx.font_rendering.cleartype_params.rendering_mode to 5 helps

Comment 7

[Bas Schouten (:bas.schouten)]

Pardon my ignorance, but where do I find jsgothic.ttc? :)

Comment 8

[Alice0775 White]

Attachment: font

Comment 9

[Bas Schouten (:bas.schouten)]

Attachment: Assume directwrite is buggy when it returns empty alpha texture bounds.

This fixes the issue, this falls back to traditional rendering when DirectWrite is returning empty bounds for the alpha texture, which seems to be indicative of this problem.

Comment 10

[Jeff Muizelaar [:jrmuizel]]

John, Bas, is there any chance we can get a reduced test case of what ever this font is doing?

Comment 11

[Jeff Muizelaar [:jrmuizel]]

(In reply to Bas Schouten (:bas) from comment #9)
> Created attachment 629680 [details] [diff] [review]
> Assume directwrite is buggy when it returns empty alpha texture bounds.
> 
> This fixes the issue, this falls back to traditional rendering when
> DirectWrite is returning empty bounds for the alpha texture, which seems to
> be indicative of this problem.

Do you know why setting gfx.content.azure.enabled to false helps?

Comment 12

[Bas Schouten (:bas.schouten)]

(In reply to Jeff Muizelaar [:jrmuizel] from comment #11)
> (In reply to Bas Schouten (:bas) from comment #9)
> > Created attachment 629680 [details] [diff] [review]
> > Assume directwrite is buggy when it returns empty alpha texture bounds.
> > 
> > This fixes the issue, this falls back to traditional rendering when
> > DirectWrite is returning empty bounds for the alpha texture, which seems to
> > be indicative of this problem.
> 
> Do you know why setting gfx.content.azure.enabled to false helps?

Yes, cairo-d2d-surface.cpp:3752 has the same logic there.

Comment 13

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 629680 [details] [diff] [review]
Assume directwrite is buggy when it returns empty alpha texture bounds.

Review of attachment 629680 [details] [diff] [review]:
-----------------------------------------------------------------

Reluctant r+ because of lack of test. Please put a reference to this bug in a comment nearby.

Comment 14

[Bas Schouten (:bas.schouten)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d846a8245199

Comment 15

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/d846a8245199

Comment 16

[Bas Schouten (:bas.schouten)]

Comment on attachment 629680 [details] [diff] [review]
Assume directwrite is buggy when it returns empty alpha texture bounds.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 715768
User impact if declined: On rare occasions missing fonts
Testing completed (on m-c, etc.): Several days m-c
Risk to taking this patch (and alternatives if risky): Very low, this is really unlikely to have a negative impact.
String or UUID changes made by this patch: None

Comment 17

[Alex Keybl [:akeybl]]

Comment on attachment 629680 [details] [diff] [review]
Assume directwrite is buggy when it returns empty alpha texture bounds.

[Triage Comment]
Low risk fix for a regression in FF15. Approved for Aurora.

Comment 18

[Bas Schouten (:bas.schouten)]

https://hg.mozilla.org/releases/mozilla-beta/rev/1513818e3c54

Comment 19

[Lukas Blakk [:lsblakk] use ?needinfo]

[Triage comment]
Updating the status flags since this has landed on Beta (15).

Comment 20

[Ioana (away)]

Verified as fixed with the reporter's test cases on:
Mozilla/5.0 (Windows NT 6.1; rv:15.0) Gecko/20100101 Firefox/15.0