Closed Bug 1202383 Opened 9 years ago Closed 9 years ago

Use build time preprocessor to enable/disable Sync on System

Categories

(Firefox OS Graveyard :: Sync, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
FxOS-S7 (18Sep)

People

(Reporter: ferjm, Assigned: ferjm)

References

Details

(Whiteboard: [partner-cherry-pick])

Attachments

(1 file)

Bug 1191770 added a mechanism for enabling/disabling features on apps. We can now use that to enable/disable Firefox Sync on the System app.
Assignee: nobody → ferjmoreno
Target Milestone: --- → FxOS-S7 (18Sep)
Summary: Use built time preprocessor to enable/disable Sync on System → Use build time preprocessor to enable/disable Sync on System
Blocks: fxos-sync
Depends on: 1191770
Attachment #8661780 - Flags: review?(etienne)
Comment on attachment 8661780 [details] [review] [gaia] ferjm:bug1202383.disablesync > mozilla-b2g:master Wow I know nothing about the build system! :) So r=me for the comments in core.js I guess, but I'm not sure who should do the real review.
Attachment #8661780 - Flags: review?(etienne)
(In reply to Etienne Segonzac (:etienne) from comment #2) > Comment on attachment 8661780 [details] [review] > [gaia] ferjm:bug1202383.disablesync > mozilla-b2g:master > > Wow I know nothing about the build system! :) Come on, don't be that modest :P
Comment on attachment 8661780 [details] [review] [gaia] ferjm:bug1202383.disablesync > mozilla-b2g:master Ricky, could you take a look at this patch, please? Thanks!
Attachment #8661780 - Flags: review?(rchien)
Comment on attachment 8661780 [details] [review] [gaia] ferjm:bug1202383.disablesync > mozilla-b2g:master LGTM
Attachment #8661780 - Flags: review?(rchien) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Blocks: 1211833
See Also: → 1211833
Whiteboard: [partner-cherry-pick]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: