Closed Bug 1451714 Opened 6 years ago Closed 5 years ago

Fix comment over LOAD_NO_NETWORK_IO flag

Categories

(Core :: Networking: Cache, task, P5)

Product:
Core
Component:
Networking: Cache
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla70
Tracking Flags:
Tracking Status
firefox61 --- wontfix
firefox70 --- fixed

People

(Reporter: mayhemer, Assigned: mayhemer)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

Bug 1451714 - Fix comment over LOAD_NO_NETWORK_IO flag, r=michal
47 bytes, text/x-phabricator-request
Details | Review 
It mentions LOAD_ONLY_FROM_CACHE flag but I think the comment is wrong.

Since we have a number of flags, with often similar names but different meanings, it's really important to document them properly.

I always rather look into the code than believing these comments.
Assignee: nobody → honzab.moz
Priority: -- → P5
Whiteboard: [necko-triaged]
Type: enhancement → task
Status: NEW → ASSIGNED

(Getting this off my list)

Pushed by honzab.moz@firemni.cz:
https://hg.mozilla.org/integration/autoland/rev/ca594da0c060
Fix comment over LOAD_NO_NETWORK_IO flag, r=michal

https://hg.mozilla.org/mozilla-central/rev/ca594da0c060

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox70: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: