1479162 - On arewexblstill page, loading the last graph makes the browser hang for 10 seconds

Status: RESOLVED WONTFIX

(Websites :: Other, defect)

Description

[Mayank Bansal]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:63.0) Gecko/20100101 Firefox/63.0
Build ID: 20180727103347

Steps to reproduce:

1. Go to https://bgrins.github.io/xbl-analysis/graph/
2. Scroll to the bottom. There is a link which says "Show lines of code per binding graph" 
3. Click on it


Actual results:

Page will hang for 10-12 seconds while the graph loads


Expected results:

not so

Profile: https://perfht.ml/2LWdzxl

Comment 1

[Mayank Bansal]

Profile with "track JIT optimization" enabled:   https://perfht.ml/2LWog31

Comment 2

[Mayank Bansal]

This is worse on Chrome

Comment 3

[Brian Grinstead [:bgrins]]

Yeah, it's really bad. It used to render by default but I made it require a click so it wouldn't slow down loading the primary graphs on the dashboard.

Thanks for posting the profile - it looks like the chart library is the culprit, which makes sense since it's rendering a huge amount of data. I guess one "fix" would be to limit the number of days that get rendered into this graph. But realistically, I'm not planning to work on that now that it doesn't slow down the page load. Patches would be welcome at https://github.com/bgrins/xbl-analysis though.

Comment 4

[Daniel Holbert [:dholbert]]

[qf-] since it sounds like this is isn't a Firefox-is-slow issue, per comment 2.

(Seems like this (not-public-facing) website is just doing a lot of computations in JS and it's slow in multiple browsers.)

Comment 5

[Kohei Yoshino [:kohei]]

The site in question is not managed by Mozilla, hence INVALID.

Comment 6

[Mayank Bansal]

As per comment 3, the slowness cannot be helped. And this is slow in other browsers also.