Remove NECKO_OFFLINE_CACHE checks

RESOLVED FIXED in mozilla9

Status

RESOLVED FIXED
7 years ago
9 months ago

People

(Reporter: matjk7, Assigned: matjk7)

Tracking

Trunk
mozilla9
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [inbound])

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Updated

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

Comment 1

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

Comment 2

7 years ago
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

7 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.

Updated

7 years ago
Assignee: nobody → matjk7

Comment 5

7 years ago
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

7 years ago
Keywords: checkin-needed

Updated

7 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: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9

Updated

9 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.