Closed Bug 1384970 Opened 7 years ago Closed 8 months ago

djvu-html5 demo spends lots of time in cross compartment wrapper overhead

Categories

(Core :: JavaScript Engine, enhancement, P3)

Product:
Core
Component:
JavaScript Engine
Type:
enhancement

Tracking

()

Status:
RESOLVED WORKSFORME
Project Flags:
Performance Impact low
Tracking Flags:
Tracking Status
firefox57 --- wontfix

People

(Reporter: ehsan.akhgari, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: perf) 

Test page: http://mateusz-matela.github.io/djvu-html5/demo.html

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

Performance is pretty abysmal, 108ms only spent in js::Wrapper::wrappedObject().  Various CCW functions also show up pretty hot.  The page seems to put all of its main code and logic in a same origin iframe, which explains the overhead we are seeing.

We should probably retest this when bug 1357862 starts to show some results.
status-firefox57: --- → wontfix
Keywords: perf
Priority: -- → P3
Whiteboard: [qf:p3]
Performance Impact: --- → P3
Whiteboard: [qf:p3]
Severity: normal → S3

Latest profile : https://share.firefox.dev/3QLsePV
Feels smooth now. Should this be closed?

Flags: needinfo?(jdemooij)

(In reply to Mayank Bansal from comment #1)

Latest profile : https://share.firefox.dev/3QLsePV
Feels smooth now. Should this be closed?

Thanks for testing. Yes let's close this - could be fixed by bug 1357862 and/or website changes (in your profile I do notice it uses a worker thread).

Status: NEW → RESOLVED
Closed: 8 months ago
Flags: needinfo?(jdemooij)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.