Closed Bug 1161677 Opened 5 years ago Closed 5 years ago

[Spark] Expose dev mode state read-only through the navigator.hasFeature() api

Categories

(Firefox OS Graveyard :: Gaia::Customizer, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

(firefox41 fixed)

RESOLVED FIXED
Tracking Status
firefox41 --- fixed

People

(Reporter: justindarc, Assigned: fabrice)

References

Details

(Whiteboard: [spark])

Attachments

(1 file)

As suggested here ( https://bugzilla.mozilla.org/show_bug.cgi?id=1160235#c5 ), we need to be able to mirror the value of the `dom.apps.developer_mode` pref to a setting that's readable by apps.
Priority: -- → P1
Whiteboard: [spark]
As per the discussion from https://bugzilla.mozilla.org/show_bug.cgi?id=1160235#c11, this use case is going to be achieved by exposing dev mode state read-only through the navigator.hasFeature() api.
Blocks: 1161157
Summary: [Spark] Need to mirror dom.apps.developer_mode pref to a read-only setting → [Spark] Expose dev mode state read-only through the navigator.hasFeature() api
Setting NI for Fabrice for his inputs on work involved for this feature. Thanks!
Flags: needinfo?(fabrice)
No longer blocks: spark-customizer-launcher
Ehsan, this is a simple mapping from a pref to a feature.  We need to expose the pref read-only to apps so we can't use the settings api for that.
Assignee: nobody → fabrice
Flags: needinfo?(fabrice)
Attachment #8607122 - Flags: review?(ehsan)
Blocks: 1166105
Comment on attachment 8607122 [details] [diff] [review]
dev-mode-feature.patch

Review of attachment 8607122 [details] [diff] [review]:
-----------------------------------------------------------------

Can you please add a test for this too?  Thanks!
Attachment #8607122 - Flags: review?(ehsan) → review+
Blocks: 1167377
https://hg.mozilla.org/mozilla-central/rev/619d60795f1b
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.