1804655 - high cpu usage and memory when using the JSON viewer on an 8MB file

Status: RESOLVED DUPLICATE of bug 1363222

(DevTools :: JSON Viewer, defect, P3)

Description

[muratkerimaslan]

Attachment: Firefox 2022-12-08 13.57 profile.json.gz

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:107.0) Gecko/20100101 Firefox/107.0

Steps to reproduce:

Opened firefox and went to "10.3.2.13:8080/database/md5sum", to see the JSON result of an API, the result is a 10 MB JSON. (Attached)

Actual results:

CPU usage of "FirefoxCP Isolated Webcontent" process, went up to 100%, and its memory usage went up 10GB in activity monitor.

Expected results:

I expected to view JSON content in either expandable tabs, or RAW JSON, but the page was unresponsive.

Comment 1

[muratkerimaslan]

Attachment: md5sum.json

The 10 MB API response that I was trying to view in the new tab

Comment 2

[muratkerimaslan]

Attachment: Screenshot 2022-12-08 at 14.01.06.png

Comment 3

[muratkerimaslan]

Attachment: Screenshot 2022-12-08 at 14.01.52.png

Comment 4

[Rares Doghi, Desktop QA]

I was able to reproduce this issue in all versions of Firefox from Esr to our latest Nightly build.
I saved the file locally and after loading it into Firefox the CPU would jump to 103% and drop back to 101%.
I will change the component to Core Dom: Content Process but I'm not entirely sure its the correct one, maybe one of our devs can take a look at the memory report and they can move it to the correct component, there is a handful of similar bugs related the FirefoxCP and im not sure if its a duplicate for any of them.

Comment 5

[Andrew McCreight [:mccr8]]

I'll make the summary more specific so it doesn't draw in comments from people with unrelated performance issues.

Comment 6

[Julian Descottes [:jdescottes]]

One option is to bail out from the json viewer and use the raw view if the size exceeds a certain limit, but we should still investigate the performance issue and see if there are obvious improvements to do.

Comment 7

[Zac Svoboda :zacnomore]

Can we combine this with bug 1363222?

Comment 8

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Yes, agree.