1451971 - Flickering navbar on ESPN

Status: VERIFIED FIXED

(Core :: Web Painting, defect)

Description

[Asif Youssuff]

Attachment: ESPN.webm

STR: 

1. Visit http://www.espn.com/nba/scoreboard
2. Hover over navbar entries (NFL/NBA/MLB etc., or subnav below - Home, Scores, Schedule, etc.)

What happens:

Entire navbar begins to flicker/disappear, turning gray

Expected result:

As when layout.display-list.retain is set to false. 

21:30.18 INFO: Narrowed inbound regression window from [6415ccbf, d5135503] (3 builds) to [6415ccbf, d112cf7b] (2 builds) (~1 steps left)
21:30.18 INFO: No more inbound revisions, bisection finished.
21:30.18 INFO: Last good revision: 6415ccbf739f36f71548540bd92c3b86ab9e1529
21:30.18 INFO: First bad revision: d112cf7b2b60e6244099dc3b599a2444ba0d1da3
21:30.18 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=6415ccbf739f36f71548540bd92c3b86ab9e1529&tochange=d112cf7b2b60e6244099dc3b599a2444ba0d1da3

Comment 1

[Matt Woodrow (:mattwoodrow)]

I can reproduce this in Nightly, but it looks to be working again in my local m-c build. Might have been fixed by bug 1451384.

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Hmm, I did a local build after that round of merges and can still reproduce on Win10. I do have WR enabled if that matters.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

Still reproduces for me on a different Win10 system as well with m-c tip and a clean profile.

Comment 4

[Matt Woodrow (:mattwoodrow)]

Ok, I fully understand this issue now and am working on a fix. I found a few unrelated, but interesting bugs in the process, lots to do!

Comment 5

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1451971 - Add new reftest for changing content within perspective without breaking sorting.

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

Comment 6

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1451971 - Rebuild the whole page when we have perspective or preserve-3d.

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

Comment 7

[Miko Mynttinen]

Comment on attachment 8966910 [details]
Bug 1451971 - Add new reftest for changing content within perspective without breaking sorting.

https://reviewboard.mozilla.org/r/235572/#review241368

Comment 8

[Miko Mynttinen]

Comment on attachment 8966911 [details]
Bug 1451971 - Rebuild the whole page when we have perspective or preserve-3d.

https://reviewboard.mozilla.org/r/235574/#review241434

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/79a5ebb7f796
Add new reftest for changing content within perspective without breaking sorting. r=miko
https://hg.mozilla.org/integration/autoland/rev/6a8df72533ba
Rebuild the whole page when we have perspective or preserve-3d. r=miko

Comment 10

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/79a5ebb7f796
https://hg.mozilla.org/mozilla-central/rev/6a8df72533ba

Comment 12

[Ryan VanderMeulen [:RyanVM]]

Looks great now!