Closed Bug 1367665 Opened 8 years ago Closed 7 years ago

[Input Latency] Firefox is 31% (53 ms) slower than Chrome in case GMail - Open Email

Categories

(Core :: JavaScript Engine, defect, P3)

defect

Tracking

()

RESOLVED INVALID
Performance Impact high

People

(Reporter: wachen, Unassigned)

References

(Blocks 1 open bug, )

Details

(Keywords: perf, Whiteboard: [QRC][QRC_Analyzed])

User Story

STR:
1. Open Gmail in logged in state.
2. Mouse click the first email in the inbox

Short Gecko Profile(cover only Input Lantency action):
https://perfht.ml/2rizrv2

Reports: https://goo.gl/pdMRw9
Notes: https://docs.google.com/spreadsheets/d/1MsTK1FW88wuLd25A18HG2KqpjYJnvrxGM9a6MeZp35w/edit#gid=256706414

Firefox Profile and Chrome Tracing will be attached in the following.
STR: 1. Open Gmail in logged in state. 2. Mouse click the first email in the inbox Reports: https://goo.gl/pdMRw9 Notes: https://docs.google.com/spreadsheets/d/1MsTK1FW88wuLd25A18HG2KqpjYJnvrxGM9a6MeZp35w/edit#gid=256706414 Firefox Profile and Chrome Tracing will be attached in the following.
User Story: (updated)
No longer depends on: 1367664
Blocks: 1367666
No longer blocks: 1367666
Firefox Profile: https://perfht.ml/2qmxr0I Chrome Tracing: https://goo.gl/gV2ExC
Whiteboard: [qf] → [qf][QRC_NeedAnalysis]
This looks like it's dominated by big blocks of js running.
Whiteboard: [qf][QRC_NeedAnalysis] → [QRC][QRC_NeedAnalysis]
User Story: (updated)
Short Gecko Profile(cover only Input Lantency action): https://perfht.ml/2rizrv2
Summary: [Input Latency] GMail - Open Email → [Input Latency] Firefox is 31% (53 ms) slower than Chrome in case GMail - Open Email
Catalin, could you help to profile this bug? Thanks.
Assignee: nobody → catalin.badea392
Flags: needinfo?(catalin.badea392)
I think someone that's more experienced with the js engine should have a look at this. The click event takes, 30ms, running js with nothing obvious showing up in the profiler. After the click event we execute a huge chunk of js ~400ms. https://perf-html.io/public/2152988fd4471adbdfafdea0d6e9d5a0d4324b5c/calltree/?hiddenThreads=&range=3.9053_4.8623&thread=5&threadOrder=0-2-3-5-1-4 The stack looks to be missing the frames calling js::RunScript. On a local profile this is executed inside a promise microtask checkpoint. Local profile: https://perf-html.io/public/cf321decb79c072d3994fa56142fe2630ee2fb5c/calltree/?hiddenThreads=&range=2.4113_2.7467&thread=4&threadOrder=0-2-3-4-1 Next, we spend time in nsRefreshDriver::DoTick, but this doesn't look out of the ordinary. https://perf-html.io/public/2152988fd4471adbdfafdea0d6e9d5a0d4324b5c/calltree/?hiddenThreads=&range=5.0565_5.1672&thread=5&threadOrder=0-2-3-5-1-4
Flags: needinfo?(catalin.badea392)
tracking under qf.
Assignee: catalin.badea392 → nobody
Component: General → JavaScript Engine
Whiteboard: [QRC][QRC_NeedAnalysis] → [qf][QRC][QRC_Analyzed]
Whiteboard: [qf][QRC][QRC_Analyzed] → [qf][QRC][QRC_Analyzed][qf:p1]
Whiteboard: [qf][QRC][QRC_Analyzed][qf:p1] → [qf:p1][QRC][QRC_Analyzed]
Whiteboard: [qf:p1][QRC][QRC_Analyzed] → [qf:p2][QRC][QRC_Analyzed]
Keywords: perf
Ted can you please profile this
Flags: needinfo?(tcampbell)
Whiteboard: [qf:p2][QRC][QRC_Analyzed] → [qf:p1][QRC][QRC_Analyzed]
Whiteboard: [qf:p1][QRC][QRC_Analyzed] → [qf:i60][qf:p1][QRC][QRC_Analyzed]
Whiteboard: [qf:i60][qf:p1][QRC][QRC_Analyzed] → [qf:f60][qf:p1][QRC][QRC_Analyzed]
Whiteboard: [qf:f60][qf:p1][QRC][QRC_Analyzed] → [qf:f61][qf:p1][QRC][QRC_Analyzed]
Flags: needinfo?(tcampbell)
Priority: -- → P3
The new Gmail UI is going to come along and probably invalidate most if not all of the analysis in here. I'm going to close this as INVALID (or maybe I should use INCOMPLETE - I don't really know), and if the new Gmail ends up being non-performant in the same way, we should file a new bug I think.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Performance Impact: --- → P1
Whiteboard: [qf:f61][qf:p1][QRC][QRC_Analyzed] → [QRC][QRC_Analyzed]
You need to log in before you can comment on or make changes to this bug.