1931514 - Make sure asyncOpenTime is not null

Status: RESOLVED FIXED

(Core :: Networking, defect, P2)

Description

[Valentin Gosu [:valentin] (he/him)]

Currently we set mAsyncOpenTime to 0 if timing isn't enabled for the channel.
For this bug we should consider not resetting the timestamp like here: https://hg.mozilla.org/try/rev/51ec9e60ae815fcfc5f2a2a845826ed7d68007f3
I'm not sure if nsITimedChannel::timingEnabled is still needed, so I'd consider removing it.

Comment 1

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1931514 - Make sure mAsycOpenTime is not null r=#necko

Comment 2

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1931514 - Remove nsITimedChannel.timingEnabled r=#necko

Timing should always be enabled.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b725bc0904ff
Make sure mAsycOpenTime is not null r=necko-reviewers,kershaw
https://hg.mozilla.org/integration/autoland/rev/169255215767
Remove nsITimedChannel.timingEnabled r=necko-reviewers,kershaw

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/b725bc0904ff
https://hg.mozilla.org/mozilla-central/rev/169255215767