Closed Bug 460328 Opened 11 years ago Closed 11 years ago

Make bypass entries a prefix match

Categories

(Core :: Networking: Cache, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: dcamp, Assigned: mayhemer)

References

()

Details

(Keywords: fixed1.9.1)

Attachments

(1 file, 1 obsolete file)

As expected, the offline spec changed online whitelist/bypass entries to be a prefix match.  This should just be a matter of removing the XXX'ed block in nsOfflineCacheDevice::GetMatchingNamespace().
Flags: blocking1.9.1?
Assignee: nobody → dcamp
Flags: blocking1.9.1? → blocking1.9.1+
Attached patch v1 (obsolete) — Splinter Review
This is just a small change in the code. The whole patch consist of a new test for whitelisting.
Assignee: dcamp → honzab
Status: NEW → ASSIGNED
Attachment #345813 - Flags: review?(dcamp)
Depends on: 443017, 461325
Blocks: 460335
Attached patch v1.1Splinter Review
Just a test changed to open a new top window instead of using iframe. Not required iframe scope patch to land this.
Attachment #345813 - Attachment is obsolete: true
Attachment #346118 - Flags: review?(dcamp)
Attachment #345813 - Flags: review?(dcamp)
Attachment #346118 - Flags: superreview?(cbiesinger)
Attachment #346118 - Flags: review?(dcamp)
Attachment #346118 - Flags: review+
Attachment #346118 - Flags: superreview?(cbiesinger) → superreview+
Landed as http://hg.mozilla.org/mozilla-central/rev/3502bada8a5c
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.