PageThumbsProtocol.js' newChannel2 implementation is slow

RESOLVED WORKSFORME

Status

()

defect
P4
normal
RESOLVED WORKSFORME
2 years ago
Last year

People

(Reporter: florian, Unassigned)

Tracking

(Blocks 1 bug)

unspecified
Points:
---
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fxperf])

Reporter

Description

2 years ago
See this profile: https://perfht.ml/2oyWRLz

It was taken on an old netbook, it's part of the profile I got when opening a new tab (so loading about:newtab for the first time), https://perfht.ml/2oyXYuH
Flags: qe-verify?
Priority: -- → P2
Whiteboard: [photon-performance]
Priority: P2 → P3
Whiteboard: [photon-performance] → [reserve-photon-performance]
Flags: qe-verify? → qe-verify-
Priority: P3 → P4
Whiteboard: [reserve-photon-performance] → [fxperf]
PageThumbsProtocol was re-implemented in native code in bug 1373258, so I doubt that the profile from comment 0 is of much use anymore. florian, have you seen this appear in profiles lately, or can I close this out as incomplete?
Flags: needinfo?(florian)
Reporter

Comment 2

Last year
(In reply to Mike Conley (:mconley) (:⚙️) (Totally backlogged on reviews and needinfos) from comment #1)
> PageThumbsProtocol was re-implemented in native code in bug 1373258, so I
> doubt that the profile from comment 0 is of much use anymore. florian, have
> you seen this appear in profiles lately, or can I close this out as
> incomplete?

Let's close this.
Status: NEW → RESOLVED
Closed: Last year
Flags: needinfo?(florian)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.