1193113 - High memory consumption on abc.net.au radio player

Status: RESOLVED WONTFIX

(Core :: General, defect)

Description

[Mark Hammond [:markh] [:mhammond]]

Sorry about the vague STR, but I felt it worth getting on file:

* Load the player http://www.abc.net.au/radio/stations/triplej/live?play=true - sadly I suspect this might not work outside of Australia.

* Check about:memory for this page - probably around 12MB.

* Leave the player running for many hours.

* Check about:memory - page will have grown to close to 1GB, Task Manager reflects this in memory used for the process (same with and without e10s, although obviously the process with the high memory usage will be different)

* "Minimize" memory has no impact - however, Shift+Reload then "Minimize" brings memory usage down to where it started.

* Leave it running long enough and Firefox will eventually die with OOM - eg, https://crash-stats.mozilla.com/report/index/c00ae7e0-dbfc-4326-8543-294e12150615, but I have many such reports if desired.

I tried on Chrome and it seems to have a much smaller leak - after 3 or so hours the memory grew to about 30MB. Today's Nightly grew to 80MB in 3 hours.

Comment 1

[Timothy Nikkel (:tnikkel)]

Can you provide a copy of the about:memory memory report?

Comment 2

[Mark Hammond [:markh] [:mhammond]]

Attachment: memory-report.json.gz

Sorry, I thought I did attach the memory report - here is one.

FTR, overnight the memory usage of the tab is sitting at 500MB.

Comment 3

[Timothy Nikkel (:tnikkel)]

This stands out to me

│    │  │  ├──296.94 MB (21.36%) -- dom
│    │  │  │  ├──296.67 MB (21.34%) ── event-targets

The mouse over of event-targets mentions xhr's. So maybe the page is leaking xhr's? Why we are worse then Chrome I don't know.

Comment 4

[Timothy Nikkel (:tnikkel)]

Also in the main process

├──265.88 MB (44.59%) ── heap-unclassified

And this is likely a different problem:

├───23.42 MB (03.93%) -- images
│   ├──19.74 MB (03.31%) -- uncached
│   │  ├──19.44 MB (03.26%) -- raster/used
│   │  │  ├──15.92 MB (02.67%) -- image(1984x128, chrome://browser/skin/[e-mail])
│   │  │  │  ├──15.92 MB (02.67%) ── source [177]

177 copies of the menu panel sprite image.

Comment 5

[Nicholas Nethercote [inactive]]

There's also this:

> │    │  └───96.42 MB (06.94%) -- js-zone(0x1139e800)
> │    │      ├──86.35 MB (06.21%) -- strings
> │    │      │  ├──46.40 MB (03.34%) -- (782 tiny)
> │    │      │  │  ├───1.64 MB (00.12%) ── string(length=3, copies=107222, "1x1")/gc-heap/latin1
> │    │      │  │  ├───1.42 MB (00.10%) ── string(length=11, copies=46630, "musicbrainz")/gc-heap/latin1
> │    │      │  │  ├───1.08 MB (00.08%) ── string(length=18, copies=35268, "MusicBrainz artist")/gc-heap/latin1
> │    │      │  │  ├───0.88 MB (00.06%) ── string(length=4, copies=57817, "16x9")/gc-heap/latin1
> │    │      │  │  ├───0.87 MB (00.06%) ── string(length=3, copies=56736, "4x3")/gc-heap/latin1
> │    │      │  │  ├───0.86 MB (00.06%) ── string(length=4, copies=56053, "Link")/gc-heap/latin1
> │    │      │  │  ├───0.86 MB (00.06%) ── string(length=7, copies=56053, "service")/gc-heap/latin1
> │    │      │  │  ├───0.68 MB (00.05%) ++ string(length=145, copies=4050, "http://resize.abcradio.net.au/jBAnpu8-0IcCI1J0SUw0KIBlURw=/340x255/center/middle/http://www.abc.net.au/radio/images/service/backgrounds/local.jpg")
> │    │      │  │  ├───0.57 MB (00.04%) ── string(length=6, copies=37541, "Artist")/gc-heap/latin1
> │    │      │  │  ├───0.52 MB (00.04%) ── string(length=7, copies=34154, "primary")/gc-heap/latin1
> │    │      │  │  ├───0.49 MB (00.04%) ++ string(length=151, copies=2923, "KLP shakes your house every Saturday night with the best party music, remixes, indie hits and classic cuts. An epic house party at yours every weekend.")
> │    │      │  │  ├───0.49 MB (00.04%) ++ string(length=25, copies=10645, "MusicBrainz release-group")
> │    │      │  │  ├───0.48 MB (00.03%) ++ string(length=197, copies=2226, "Whether it be at festivals, in stadiums or sweaty pubs, in our own studios or even on our rooftop, triple j has captured some of the best live music performances that have happened in this country.")
> │    │      │  │  ├───0.47 MB (00.03%) ++ string(length=30, copies=10280, "/api/v1/plays/triplej/now.json")
> │    │      │  │  ├───0.35 MB (00.03%) ── string(length=7, copies=22849, "Artwork")/gc-heap/latin1
> │    │      │  │  ├───0.34 MB (00.02%) ── string(length=9, copies=11118, "unearthed")/gc-heap/latin1
> │    │      │  │  ├───0.31 MB (00.02%) ── string(length=22, copies=10281, "music.abcradio.net.au:")/gc-heap/latin1
> │    │      │  │  ├───0.29 MB (00.02%) ++ string(length=52, copies=3796, "rtmp://cp112896.live.edgefcs.net/live/triple_j@40677")
> │    │      │  │  ├───0.29 MB (00.02%) ++ string(length=59, copies=3796, "http://www.abc.net.au/res/streaming/audio/hls/triple_j.m3u8")
> │    │      │  │  ├───0.29 MB (00.02%) ++ string(length=61, copies=3796, "http://www.abc.net.au/res/streaming/audio/windows/triplej.asx")
> │    │      │  │  ├───0.28 MB (00.02%) ── string(length=7, copies=18447, "Release")/gc-heap/latin1
> │    │      │  │  ├───0.27 MB (00.02%) ── string(length=16, copies=8984, "Unearthed artist")/gc-heap/latin1
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=145, copies=1470, "http://resize.abcradio.net.au/PK9JyJqcBmtpRdUnTKoaajaSBUs=/100x75/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/84VLhP3sF4tS6CUgn2mvSVzuZqQ=/160x120/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/8nlbn6KZFDuKpEvz0gWujRPDNgM=/580x326/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/QWX_kYspRE4lvUnLXEWIe4oWu7A=/940x940/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/US4uqQvfFvi5SjenRG-vGafm1sg=/340x255/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/W9NZQFz_64kfHCBGWnU4AO2E8ws=/580x580/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/ZDVhBZAL_t4GTsJJOM-UkbjKBGw=/340x340/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/bl5YIhzOq_jo-hm_Dl0f7yk1iQc=/160x160/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/k66xdlF6HBNzvHIlN-L63NZrIfQ=/100x100/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/wI59XAAGn9rgnREjA_yuqLx35OQ=/940x529/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=146, copies=1470, "http://resize.abcradio.net.au/wbyj0W0SsvYyQ2mNxuPpAWPfuCk=/220x124/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.25 MB (00.02%) ++ string(length=147, copies=1470, "http://resize.abcradio.net.au/aNwnk1Mj0RsBbfbmsiMii68a0-I=/1300x731/right/middle/http://www.abc.net.au/radio/images/presenter/kristy_lee_peters.jpg")
> │    │      │  │  ├───0.23 MB (00.02%) ── string(length=10, copies=7594, "pijKWndQGE")/gc-heap/latin1

And so on, and so on. The notable thing here is that there are many strings
duplicated thousands of times. Very suspicious to have 2923 copies of a string
like "KLP shakes your house every Saturday night with the best party music,
remixes, indie hits and classic cuts. An epic house party at yours every
weekend."

It definitely feels like the site is at fault, accumulating data and never
letting it go.

Comment 6

[Mark Hammond [:markh] [:mhammond]]

Exactly the same behaviour with Chrome, so this is a site issue.

Comment 7

[Timothy Nikkel (:tnikkel)]

(In reply to Timothy Nikkel (:tnikkel) from comment #4)
> And this is likely a different problem:
> 
> ├───23.42 MB (03.93%) -- images
> │   ├──19.74 MB (03.31%) -- uncached
> │   │  ├──19.44 MB (03.26%) -- raster/used
> │   │  │  ├──15.92 MB (02.67%) -- image(1984x128,
> chrome://browser/skin/[e-mail])
> │   │  │  │  ├──15.92 MB (02.67%) ── source [177]
> 
> 177 copies of the menu panel sprite image.

I think this got fixed in another bug (don't have the bug number handy). Had to do with css variables.