1375795 - gradient render problem in new version 54.0

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[damien.mordaque]

Attachment: before and after the new version bug.png

User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:54.0) Gecko/20100101 Firefox/54.0
Build ID: 20170608105825

Steps to reproduce:

CSS gradient in balise tr of table not render correctly since the new version 54.0


Actual results:

The render is too bad ! there is a gradient on background of all the page and when we scroll or open popin the problem disappear

Comment 1

[Pascal Chevrel:pascalc]

Damien, could you provide the url of the page where you see this bug or a reduced testcase so as that others can reproduce your bug? Thanks

Comment 2

[damien.mordaque]

Attachment: To reproduce bug

A page to reproduce the bug. I found that is the css   " background-attachment: fixed; " the cause of bug. 

The css command have not the same render in new released.

Comment 3

[Alice0775 White]

This seems to be already fixed on Beta55 by Bug 929484.

@Damien,
Could you test this with Beta55[1]?
(ATTENTION, Please test with new profile.)

[1]https://www.mozilla.org/en-US/firefox/channel/desktop/

Comment 4

[Astley Chen (inactive)]

(In reply to Alice0775 White from comment #3)
> This seems to be already fixed on Beta55 by Bug 929484.
> 
> @Damien,
> Could you test this with Beta55[1]?
> (ATTENTION, Please test with new profile.)
> 
> [1]https://www.mozilla.org/en-US/firefox/channel/desktop/

Alice, do you have the site url for verification?
Bug 929484 was to separate table frame rendering into smaller ones(ie, by cells) to enhance our rendering performance. I'm not sure why it could fix the bug, but a test url will help to figure out.

Comment 5

[Alice0775 White]

(In reply to Astley Chen [:astley] (UTC-7) from comment #4)
> (In reply to Alice0775 White from comment #3)
> > This seems to be already fixed on Beta55 by Bug 929484.
> > 
> > @Damien,
> > Could you test this with Beta55[1]?
> > (ATTENTION, Please test with new profile.)
> > 
> > [1]https://www.mozilla.org/en-US/firefox/channel/desktop/
> 
> Alice, do you have the site url for verification?
> Bug 929484 was to separate table frame rendering into smaller ones(ie, by
> cells) to enhance our rendering performance. I'm not sure why it could fix
> the bug, but a test url will help to figure out.

I am not sure site url, but test case is attachment 8881217 [details] of Comment 2.

Comment 6

[damien.mordaque]

It's ok with the Beta55. 
When this released will be post ?

Comment 7

[Astley Chen (inactive)]

(In reply to damien.mordaque from comment #6)
> It's ok with the Beta55. 
> When this released will be post ?

2017-08-08

Comment 8

[Astley Chen (inactive)]

Morris, a heads-up for you and see if you have any other idea regards to how your patch fix this issue in bug 929484.

Comment 9

[Morris Tseng [:mtseng] [:Morris] (Inactive)]

I guess we have some problem with old code when rendering background-attachment:fixed. Bug 929484 rewrite the mechanism of rendering background. So, this should fix the problem.

Comment 10

[Astley Chen (inactive)]

(In reply to Morris Tseng [:mtseng] [:Morris] from comment #9)
> I guess we have some problem with old code when rendering
> background-attachment:fixed. Bug 929484 rewrite the mechanism of rendering
> background. So, this should fix the problem.

kill two birds with one stone. Thanks for feedback!