Closed
Bug 678794
Opened 12 years ago
Closed 12 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•12 years ago
|
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #552942 -
Flags: review?(jduell.mcbugs)
Comment 2•12 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•12 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•12 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•12 years ago
|
Assignee: nobody → matjk7
Comment 5•12 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•12 years ago
|
Keywords: checkin-needed
Updated•12 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment 6•12 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/8fbc35ff4d7c
Keywords: checkin-needed
Whiteboard: [inbound]
Comment 7•12 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/8fbc35ff4d7c
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•