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)
Core
JavaScript Engine
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.
Reporter | ||
Comment 1•8 years ago
|
||
Firefox Profile:
https://perfht.ml/2qmxr0I
Chrome Tracing:
https://goo.gl/gV2ExC
Updated•7 years ago
|
Whiteboard: [qf] → [qf][QRC_NeedAnalysis]
Comment 2•7 years ago
|
||
This looks like it's dominated by big blocks of js running.
Updated•7 years ago
|
Whiteboard: [qf][QRC_NeedAnalysis] → [QRC][QRC_NeedAnalysis]
Short Gecko Profile(cover only Input Lantency action):
https://perfht.ml/2rizrv2
Updated•7 years ago
|
Summary: [Input Latency] GMail - Open Email → [Input Latency] Firefox is 31% (53 ms) slower than Chrome in case GMail - Open Email
Comment 5•7 years ago
|
||
Catalin, could you help to profile this bug? Thanks.
Assignee: nobody → catalin.badea392
Flags: needinfo?(catalin.badea392)
Comment 6•7 years ago
|
||
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)
Comment 7•7 years ago
|
||
tracking under qf.
Assignee: catalin.badea392 → nobody
Component: General → JavaScript Engine
Whiteboard: [QRC][QRC_NeedAnalysis] → [qf][QRC][QRC_Analyzed]
Updated•7 years ago
|
Whiteboard: [qf][QRC][QRC_Analyzed] → [qf][QRC][QRC_Analyzed][qf:p1]
Updated•7 years ago
|
Whiteboard: [qf][QRC][QRC_Analyzed][qf:p1] → [qf:p1][QRC][QRC_Analyzed]
Updated•7 years ago
|
Whiteboard: [qf:p1][QRC][QRC_Analyzed] → [qf:p2][QRC][QRC_Analyzed]
Comment 8•7 years ago
|
||
Ted can you please profile this
Flags: needinfo?(tcampbell)
Whiteboard: [qf:p2][QRC][QRC_Analyzed] → [qf:p1][QRC][QRC_Analyzed]
Updated•7 years ago
|
Whiteboard: [qf:p1][QRC][QRC_Analyzed] → [qf:i60][qf:p1][QRC][QRC_Analyzed]
Updated•7 years ago
|
Whiteboard: [qf:i60][qf:p1][QRC][QRC_Analyzed] → [qf:f60][qf:p1][QRC][QRC_Analyzed]
Updated•7 years ago
|
Whiteboard: [qf:f60][qf:p1][QRC][QRC_Analyzed] → [qf:f61][qf:p1][QRC][QRC_Analyzed]
Updated•7 years ago
|
Flags: needinfo?(tcampbell)
Updated•7 years ago
|
Priority: -- → P3
Comment 9•7 years ago
|
||
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
Updated•3 years ago
|
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.
Description
•