1897995 - on https://in.investing.com/charts/live-charts, the charts appear to be completely blank

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect)

Description

[Alice0775 White]

This is clone of Bug 1897395 comment#2

STR:
1 Open https://in.investing.com/charts/live-charts (no need logged in)

AR:
the charts appear to be completely blank

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=14d68154a5bee022c96b908cfcf492dcb4f7f085&tochange=7cdb37e233eb8ff913e8aa2214f90659a7e261c3

NOTE,
However, I cannot reproduce the issue on Firefox126.0 and 127.0b4.
So, this seems to be nightly build only issue. Setting layout.tables.scrollable-cells to false fixes the issue on Nightly128.0a1.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

:emilio, since you are the author of the regressor, bug 1880550, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 2

[Daniel Holbert [:dholbert]]

(In reply to Alice0775 White from comment #0)

So, this seems to be nightly build only issue. Setting layout.tables.scrollable-cells to false fixes the issue on Nightly128.0a1.

Just for release-management purposes: note that the relevant pref here, layout.tables.scrollable-cells, is default-false on release/beta for now (as of bug 1877719), until we make a decision to change that.

So: even though this is a regression on Nightly, we don't need to worry about this regression riding the trains to release.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jfkthame, could you have a look please?

For more information, please visit BugBot documentation.

Comment 4

[Daniel Holbert [:dholbert]]

Severity is tricky here; if this were on the way to release, it'd be clearly s2, but given that we have this behind a pref, let's say s3.

In any case, we'll need to fix this before the pref rides the trains.

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1880550

Comment 6

[Liz Henry (:lizzard) (relman/hg->git project)]

Is there a bug this should be blocking, related to the pref riding the trains?

Comment 7

[Daniel Holbert [:dholbert]]

(In reply to Liz Henry (:lizzard) (relman/hg->git project) from comment #6)

Is there a bug this should be blocking, related to the pref riding the trains?

There is now; I've just filed bug 1904159 to cover this.

Comment 8

[Daniel Holbert [:dholbert]]

(In reply to Daniel Holbert [:dholbert] from comment #4)

Severity is tricky here; if this were on the way to release, it'd be clearly s2, but given that we have this behind a pref, let's say s3.

In any case, we'll need to fix this before the pref rides the trains.

Update: we turned off the pref for Nightly as well (bug 1877719 comment 20 - 22), so this now doesn't reproduce on any of our release channels by default, and I think we can consider this bug FIXED by that change (just as we considered the similar YouTube issue in bug 1877719 to be fixed by that change).

I'll add a note on the relevant bugs tracking the pref-reenabling to be sure we don't regress this at that point.

Comment 9

[Bianca Hidecuti, Desktop Test Engineering [:bhidecuti]]

I was able to reproduce the issue on an affected Firefox Nightly build from 2024-05-21 on Windows 11, while following the steps described in Comment 0.
Verified as fixed using Firefox 130.0b5 and also the latest 131.0a1 Nightly, on Windows 11, MacOS 14.4 and Ubuntu 22.04. The chart from https://in.investing.com/charts/live-charts is no longer blank and the layout.tables.scrollable-cells pref is set to false by default.