The default bug view has changed. See this FAQ.

Remove NECKO_OFFLINE_CACHE checks

RESOLVED FIXED in mozilla9

Status

()

Core
Build Config
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Matheus Kerschbaum, Assigned: Matheus Kerschbaum)

Tracking

Trunk
mozilla9
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound])

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Updated

6 years ago
Depends on: 675500, 678792
(Assignee)

Comment 1

6 years ago
Created attachment 552942 [details] [diff] [review]
patch
Attachment #552942 - Flags: review?(jduell.mcbugs)
Michal,

This seems fine to me.  Can you think of any reason we'd want to keep the offline cache code optional at compilation time?  If not I'll +r this.
I don't see any reason to keep it. Maybe Honza?

Btw. we have the same compile time option for the disk cache which could be IMO removed too.
(Assignee)

Comment 4

6 years ago
(In reply to Michal Novotny (:michal) from comment #3)
> Btw. we have the same compile time option for the disk cache which could be
> IMO removed too.

That is bug 678792.
Assignee: nobody → matjk7
Comment on attachment 552942 [details] [diff] [review]
patch

If Michal's good with this, then let's do it.
Attachment #552942 - Flags: review?(jduell.mcbugs) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed

Updated

6 years ago
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
http://hg.mozilla.org/integration/mozilla-inbound/rev/8fbc35ff4d7c
Keywords: checkin-needed
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/8fbc35ff4d7c
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.