1363483 - Table painting busted after Bug 929484

Status: VERIFIED FIXED

(Core :: Layout, defect)

Description

[Trevor Rowbotham [:rowbot]]

Attachment: 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

Comment 1

[Timothy Nikkel (:tnikkel)]

[Tracking Requested - why for this release]:

Comment 2

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

I'm going to handle this.

Comment 3

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

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.

Comment 4

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

I can reproduce 100% now when I resize my firefox. I'll debug this.

Comment 5

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

Attachment: test-table-resize.html

Reduced test case. 

Steps to reproduce:
1. Open the html.
2. Resize the window.

Comment 6

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

Attachment: Bug 1363483 - Add nsDisplayTableBackgroundColor to avoid display item collision when processing background color of table.

Review commit: https://reviewboard.mozilla.org/r/137860/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/137860/

Comment 7

[Matt Woodrow (:mattwoodrow)]

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

Comment 8

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

Try looks good.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/efdda23fd13d
Add nsDisplayTableBackgroundColor to avoid display item collision when processing background color of table. r=mattwoodrow

Comment 10

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

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

Comment 12

[Trevor Rowbotham [:rowbot]]

I can confirm this is fixed with the 2017-05-12 nightly. Thanks!