Closed
Bug 678794
Opened 15 years ago
Closed 15 years ago
Remove NECKO_OFFLINE_CACHE checks
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla9
People
(Reporter: matjk7, Assigned: matjk7)
References
Details
(Whiteboard: [inbound])
Attachments
(1 file)
|
19.03 KB,
patch
|
jduell.mcbugs
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Flags: in-testsuite-
|
Assignee | |
Updated•15 years ago
|
|
|
Assignee | |
Comment 1•15 years ago
|
||
Attachment #552942 -
Flags: review?(jduell.mcbugs)
|
||
Comment 2•15 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.
|
||
Comment 3•15 years ago
|
||
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•15 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•15 years ago
|
Assignee: nobody → matjk7
|
|
||
Comment 5•15 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•15 years ago
|
Keywords: checkin-needed
|
||
Updated•15 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
|
|
||
Comment 6•15 years ago
|
||
Keywords: checkin-needed
Whiteboard: [inbound]
|
|
||
Comment 7•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
|
||
Updated•8 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•