Closed
Bug 748240
Opened 13 years ago
Closed 13 years ago
"view-source:" doesn't load HTML from AppCache
Categories
(Core :: Networking: Cache, defect)
Core
Networking: Cache
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: timdream, Assigned: mayhemer)
References
Details
Attachments
(1 file)
(deleted),
patch
|
bzbarsky
:
review+
mayhemer
:
checkin+
|
Details | Diff | Splinter Review |
Step to reproduce:
1. Go to a AppCache enabled website, make sure it's cached
2. Verified it is AppCache'd by checking about:cache?device=offline
3. Delete the HTTP cache (Preference -> Advanced -> Network -> Cached Web Content -> Clear Now)
4. Go offline (File -> Work Offline)
5. Go back to the website, it should load from AppCache
6. View source (cmd+U)
Expected result:
* view the source code
Actual result:
* Problem loading page
==
It's very tricky to reproduce the behavior in Firefox, but in B2G Gaia we test our apps by inserting offline AppCache directly into the profile with a fictitious hostname, that makes the behavior easy to surface (with the view source hardware key from in our dev preview phone)
Is it possible to make view-source to access not only HTTP cache but also AppCache?
Reporter | ||
Updated•13 years ago
|
Component: Networking: Cache → View Source
Product: Core → Toolkit
QA Contact: networking.cache → view.source
Comment 1•13 years ago
|
||
No, this was in the right place. Necko is what's not hitting the appcache here. The toolkit code is just using the APIs core exposes for this stuff.
Component: View Source → Networking: Cache
Product: Toolkit → Core
QA Contact: view.source → networking.cache
Reporter | ||
Comment 2•13 years ago
|
||
(In reply to Boris Zbarsky (:bz) from comment #1)
> No, this was in the right place. Necko is what's not hitting the appcache
> here. The toolkit code is just using the APIs core exposes for this stuff.
Oh thanks.
Assignee | ||
Comment 3•13 years ago
|
||
I'll look at this. I suspect this may be complicated, though...
Assignee: nobody → honzab.moz
Assignee | ||
Comment 4•13 years ago
|
||
It was damn easy! :) Just let nsViewSourceChannel implement and forward nsIApplicatonCacheChannel.
Attachment #623113 -
Flags: review?(bzbarsky)
Assignee | ||
Updated•13 years ago
|
Status: NEW → ASSIGNED
Comment 5•13 years ago
|
||
Comment on attachment 623113 [details] [diff] [review]
v1
r=me
Attachment #623113 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 6•13 years ago
|
||
Comment on attachment 623113 [details] [diff] [review]
v1
https://hg.mozilla.org/integration/mozilla-inbound/rev/c5023518db2f
Attachment #623113 -
Flags: checkin+
Comment 7•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
Comment 8•13 years ago
|
||
This patch may have triggered a new intermittent leak in mochitests (bug 754588). Any thoughts on whether or how that might be related?
You need to log in
before you can comment on or make changes to this bug.
Description
•