Table painting busted after Bug 929484

VERIFIED FIXED in Firefox 55

Status

()

Core
Layout
VERIFIED FIXED
5 months ago
5 months ago

People

(Reporter: Trevor Rowbotham, Assigned: mtseng)

Tracking

({regression})

Trunk
mozilla55
regression
Points:
---

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox53 unaffected, firefox54 unaffected, firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(3 attachments)

(Reporter)

Description

5 months ago
Created attachment 8866010 [details]
table_layout.JPG

STR:
1) Open a new tab.
2) Navigate to [1]. Observe broken painting.

This occurs approximately 90% of the time for me. It seems to occur more frequently in a newly opened tab, so if it doesn't do it the first time, try closing the tab and opening a new one and repeating the STR.

Refreshing the page, switching tabs, or resizing the window seems to fix the problem.


Last good revision: 33df8c04309cf792b214c5c6c903a391048f5516
First bad revision: 2a15fd71cf351e12c1286139e53b42b4cc486141
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=33df8c04309cf792b214c5c6c903a391048f5516&tochange=2a15fd71cf351e12c1286139e53b42b4cc486141

[1] https://bugs.php.net/search.php?limit=30&order_by=id&direction=DESC&cmd=display&status=Open&bug_type=All
[Tracking Requested - why for this release]:
Blocks: 929484
status-firefox55: --- → affected
Flags: needinfo?(mtseng)
(Assignee)

Comment 2

5 months ago
I'm going to handle this.
Assignee: nobody → mtseng
Flags: needinfo?(mtseng)
(Assignee)

Comment 3

5 months ago
I can reproduce it in my local. But the occur rate is not very high. I only see 2 times in my local. I'll find more reliable way to reproduce this.
(Assignee)

Comment 4

5 months ago
I can reproduce 100% now when I resize my firefox. I'll debug this.
(Assignee)

Comment 5

5 months ago
Created attachment 8866174 [details]
test-table-resize.html

Reduced test case. 

Steps to reproduce:
1. Open the html.
2. Resize the window.
Comment hidden (mozreview-request)

Comment 7

5 months ago
mozreview-review
Comment on attachment 8866230 [details]
Bug 1363483 - Add nsDisplayTableBackgroundColor to avoid display item collision when processing background color of table.

https://reviewboard.mozilla.org/r/137860/#review141420
Attachment #8866230 - Flags: review?(matt.woodrow) → review+
(Assignee)

Comment 8

5 months ago
Try looks good.

Comment 9

5 months ago
Pushed by mtseng@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/efdda23fd13d
Add nsDisplayTableBackgroundColor to avoid display item collision when processing background color of table. r=mattwoodrow
https://hg.mozilla.org/mozilla-central/rev/efdda23fd13d
Status: NEW → RESOLVED
Last Resolved: 5 months ago
status-firefox55: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55

Updated

5 months ago
Duplicate of this bug: 1364186
status-firefox53: --- → unaffected
status-firefox54: --- → unaffected
status-firefox-esr52: --- → unaffected
Keywords: regression
(Reporter)

Comment 12

5 months ago
I can confirm this is fixed with the 2017-05-12 nightly. Thanks!
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.