Remove parts of bug 1312754 and bug 1348061

RESOLVED FIXED in Firefox 55

Status

()

enhancement
P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mayhemer, Assigned: mayhemer)

Tracking

Trunk
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

(Whiteboard: [necko-active])

Attachments

(1 attachment)

Assignee

Description

2 years ago
Based on bug 1362071 comment 1.

Only left should be:
- nsHttpHandler::ThrottleTransaction down to nsHttpConnectionMgr::OnMsgThrottleTransaction
- preferences in all.js
- nsHttpChannel::OnClassOfServiceUpdated() calling only gHttpHandler->ThrottleTransaction
- nsHttpTransaction::mThrottleResponse


Everything else introduced in these two bugs can go.
Assignee

Updated

2 years ago
Blocks: 1365307
Assignee

Updated

2 years ago
Blocks: 1360793
Whiteboard: [necko-backlog]
Assignee

Updated

2 years ago
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Whiteboard: [necko-backlog] → [necko-active]
Assignee

Comment 2

2 years ago
Comment on attachment 8869224 [details] [diff] [review]
v1 (remove Throttling service et al)

Nick, I'm sorry to remove your child, but it turns out we need a more sophisticated mechanism (being born in bug 1365307).

This patch only leaves few bits which that bug may use (including all.js preferences).

Thanks.
Attachment #8869224 - Attachment description: v1 [builds] → v1 (remove Throttling service et al)
Attachment #8869224 - Flags: review?(hurley)

Comment 3

2 years ago
Comment on attachment 8869224 [details] [diff] [review]
v1 (remove Throttling service et al)

Review of attachment 8869224 [details] [diff] [review]:
-----------------------------------------------------------------

Which child?! I have more than... oooooooooh, you mean code I wrote! :)

The only good code is removed code!
Attachment #8869224 - Flags: review?(hurley) → review+
Assignee

Comment 4

2 years ago
(In reply to Nicholas Hurley [:nwgh][:hurley] (also hurley@todesschaf.org) from comment #3)
> Comment on attachment 8869224 [details] [diff] [review]
> v1 (remove Throttling service et al)
> 
> Review of attachment 8869224 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Which child?! I have more than... oooooooooh, you mean code I wrote! :)
> 
> The only good code is removed code!

:D thanks!

Comment 6

2 years ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e4d329c85dfb
Remove net::ThrottlingService, don't suspend/resume download channels on page load. r=hurley
Keywords: checkin-needed

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e4d329c85dfb
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee

Updated

2 years ago
Blocks: 1360865
You need to log in before you can comment on or make changes to this bug.