802623 - Give Update Ping Low Network Priority

Status: NEW

(Toolkit :: Application Update, defect, P3)

Description

[Patrick McManus [:mcmanus]]

I hope to make channel priorities a more useful attribute in necko so I'm cleaning up the priority some resource requests are classified with.

Right now safe update pings are transferred with default priority, this patch changes it to low priority suitable to a background process.

Comment 1

[Patrick McManus [:mcmanus]]

Attachment: patch 0

Comment 2

[Robert Strong (they/them - no direct email)]

With app update being one of the pieces that if it breaks people don't get updated I'd prefer holding off on changing this until most if not all other possible JS candidates for this have landed and baked. Are there other JS candidates?

Comment 3

[Patrick McManus [:mcmanus]]

(In reply to Robert Strong [:rstrong] (do not email) from comment #2)
> With app update being one of the pieces that if it breaks people don't get
> updated I'd prefer holding off on changing this until most if not all other
> possible JS candidates for this have landed and baked. Are there other JS
> candidates?

just 802679

Comment 4

[Robert Strong (they/them - no direct email)]

Comment on attachment 672314 [details] [diff] [review]
patch 0

If force is true then it is a foreground check and the priority shouldn't be changed. r=me with that