Closed Bug 1509823 Opened 6 years ago Closed 5 years ago

[socket process] Fix HTTP cache data writing

Categories

(Core :: Networking: HTTP, enhancement, P2)

enhancement

Tracking

()

RESOLVED INVALID
Tracking Status
firefox65 --- affected

People

(Reporter: dragana, Assigned: kershaw)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

      No description provided.
Whiteboard: [necko-triaged]

Please coordinate with bug 1497245.

Assignee: nobody → kershaw
Status: NEW → ASSIGNED

I check our current code again. We did already send the data to parent process at [1].
To see if cache is really working, I also did a simple local test.
Test steps:

  1. Open network panel.
  2. Go to example.com.
  3. Load example.com again and see if the response is from cache.

The result shows that cache should be working.

Dragana, do you think are there other things we should do for cache?

[1] https://hg.mozilla.org/projects/larch/file/tip/netwerk/protocol/http/HttpTransactionChild.cpp#l342

Flags: needinfo?(dd.mozilla)

(In reply to Kershaw Chang [:kershaw] from comment #2)

I check our current code again. We did already send the data to parent process at [1].
To see if cache is really working, I also did a simple local test.
Test steps:

  1. Open network panel.
  2. Go to example.com.
  3. Load example.com again and see if the response is from cache.

The result shows that cache should be working.

Dragana, do you think are there other things we should do for cache?

[1] https://hg.mozilla.org/projects/larch/file/tip/netwerk/protocol/http/HttpTransactionChild.cpp#l342

no. thanks for checking.

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Flags: needinfo?(dd.mozilla)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.