HWA causes rendering issue on http://dogtime.com/

RESOLVED FIXED in mozilla16

Status

()

Core
Layout
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: geeknik, Assigned: bas)

Tracking

Trunk
mozilla16
x86_64
Windows 7
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(4 attachments)

(Reporter)

Description

5 years ago
When using the latest 64-bit Nightly Firefox on 64-bit Windows 7 Pro SP1 with HWA enabled, there is a rendering issue on http://dogtime.com/puppy-training-what-to-chew-dunbar.html. The page loads and renders fine, but when you start scrolling down, the white background disappears. If you disable HWA, this doesn't happen.


about:buildconfig
Built from http://hg.mozilla.org/mozilla-central/rev/d9d61d199b11

about:graphics

Adapter Description NVIDIA GeForce GTX 570
Vendor ID 0x10de
Device ID 0x1081
Adapter RAM 1279
Adapter Drivers nvd3dumx,nvwgf2umx,nvwgf2umx nvd3dum,nvwgf2um,nvwgf2um
Driver Version 9.18.13.448
Driver Date 6-11-2012
Direct2D Enabled true
DirectWrite Enabled true (6.1.7601.17789)
ClearType Parameters DISPLAY1 [ Gamma: 2200 Pixel Structure: RGB ClearType Level: 100 Enhanced Contrast: 100 ] DISPLAY2 [ Gamma: 2200 Pixel Structure: RGB ClearType Level: 100 Enhanced Contrast: 200 ] 
WebGL Renderer Google Inc. -- ANGLE (NVIDIA GeForce GTX 570) -- OpenGL ES 2.0 (ANGLE 1.0.0.1041)
GPU Accelerated Windows 0
(Reporter)

Comment 1

5 years ago
Quite possible a dupe of https://bugzilla.mozilla.org/show_bug.cgi?id=770002.

Comment 2

5 years ago
When I disable Azure Content Acceleration from about:config, it is no longer reproducible.
To be exact gfx.content.azure.enabled changed to false.
(Assignee)

Updated

5 years ago
Assignee: nobody → bas.schouten
Blocks: 715768
tracking-firefox15: --- → ?

Updated

5 years ago
Duplicate of this bug: 770036

Comment 4

5 years ago
reduced html
attachment 638208 [details]
(Assignee)

Comment 5

5 years ago
Alice, any idea if this regressed last night? This could be a regression of DLBI combined with Azure-content.

Comment 6

5 years ago
(In reply to Bas Schouten (:bas) from comment #5)
> Alice, any idea if this regressed last night? This could be a regression of
> DLBI combined with Azure-content.

No, this is related to DLBI, I think.

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 Bug 715768
(Assignee)

Comment 7

5 years ago
Created attachment 638238 [details]
Reduced testcase

This very basic page already shows rendering issues when scrolling.
(Assignee)

Comment 8

5 years ago
I've tracked the root cause of this bug to a problem with partial uploading not correctly dealing with extend modes. I'm working on a solution to the problem. Definitely a regression from content Azure.
Status: NEW → ASSIGNED
[triage comment]
Adding qawanted to confirm if this is only a Firefox 64 bit issue or a windows 64 bit issue (with 32 bit Firefox as well).
tracking-firefox15: ? → +
Keywords: qawanted

Comment 10

5 years ago
I have Windows 7 Pro SP1 64-bit and 32-bit Firefox Nightly.
(Assignee)

Comment 11

5 years ago
Created attachment 638562 [details] [diff] [review]
Part 1: Fix bug in SSE2 shuffling for horizontal scaling.
Attachment #638562 - Flags: review?(jmuizelaar)
(Assignee)

Comment 12

5 years ago
Created attachment 638563 [details] [diff] [review]
Part 2: Adjust test to test for ordering in horizontal halfscaling
Attachment #638563 - Flags: review?(jmuizelaar)
Comment on attachment 638562 [details] [diff] [review]
Part 1: Fix bug in SSE2 shuffling for horizontal scaling.

Please provide a test case of some kind.
Attachment #638562 - Flags: review?(jmuizelaar) → review+
(Assignee)

Comment 14

5 years ago
Created attachment 638567 [details] [diff] [review]
Part 3: Adjust partial bitmap uploading code to deal with extend modes.

This patch is the real 'fix' to this bug, it actually lets the partial uploading code deal with extend codes. I added some special cases for where we could still partially upload to try and get optimal performance & quality. In theory for REFLECT extend mode we could try a little harder but that seems like overkill to me.
Attachment #638567 - Flags: review?(jmuizelaar)
Attachment #638563 - Flags: review?(jmuizelaar) → review+
Comment on attachment 638567 [details] [diff] [review]
Part 3: Adjust partial bitmap uploading code to deal with extend modes.

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

::: gfx/2d/DrawTargetD2D.cpp
@@ +2327,5 @@
>  
>    IntSize size = aSurface->GetSize();
>  
>    Rect uploadRect(0, 0, size.width, size.height);
>  

// Limit the uploadRect as much as possible without supporting discontiguous uploads 
//
//                               region we will paint from
//   uploadRect
//   .---------------.              .---------------.         resulting uploadRect
//   |               |rect          |               |
//   |          .---------.         .----.     .----.          .---------------.
//   |          |         |  ---->  |    |     |    |   ---->  |               |
//   |          '---------'         '----'     '----'          '---------------'
//   '---------------'              '---------------'
//
//

@@ +2340,5 @@
> +    uploadRect = uploadRect.Intersect(rect);
> +
> +    // We now proceed to check if we can limit at least one dimension of the
> +    // upload rect safely without looking at extend mode.
> +  } else if (rect.x >= 0 && (rect.width + rect.x) < size.width) {

rect.XMost() perhaps?
Attachment #638567 - Flags: review?(jmuizelaar) → review+
https://hg.mozilla.org/mozilla-central/rev/de7ab39f550d
https://hg.mozilla.org/mozilla-central/rev/a5b33148ae58
https://hg.mozilla.org/mozilla-central/rev/94d461c799fd
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
I believe comment 10 satisfies the qawanted ask on this bug. Please re-add if not. Otherwise, we are tracking for verification using the tracking/status flags.
Keywords: qawanted
[Triage Comment]

If this can be uplifted to Beta 15 please nominate for approval.
(Assignee)

Comment 19

5 years ago
Fixed on beta by turning off Azure content:

https://hg.mozilla.org/releases/mozilla-beta/rev/5f2f7027b65b
tracking-firefox15: + → ---

Updated

4 years ago
Depends on: 849033
You need to log in before you can comment on or make changes to this bug.