Closed Bug 1361691 Opened 3 years ago Closed 3 years ago

Skip URI classifier if channel is marked with urgent-start

Categories

(Core :: Networking: HTTP, enhancement)

enhancement
Not set

Tracking

()

RESOLVED INVALID

People

(Reporter: kershaw, Unassigned)

References

(Blocks 1 open bug)

Details

Urgent-start flag should be only marked for channels which are triggered by user interaction. So, I think we can skip URI classifier and directly jump to connect.
(In reply to Kershaw Chang [:kershaw] from comment #0)
> Urgent-start flag should be only marked for channels which are triggered by
> user interaction. So, I think we can skip URI classifier and directly jump
> to connect.

Does this sound reasonable to you, Honza?
Flags: needinfo?(honzab.moz)
Absolutely not.  Only indication of not doing classification on the URI is missing LOAD_CLASSIFY_URI flag - not something urgent-start or any other class should have an effect on.  Generally - anything else MUST NOT affect classification.  Every navigation or load must be classified unless explicitly turned off by the user.

I would not give the urgent-start flag that big weight.  It's a nice to have optimization that may help under a heavy utilization of traffic to the same origin, but it doesn't mean anything else.
Status: NEW → RESOLVED
Closed: 3 years ago
Flags: needinfo?(honzab.moz)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.