Closed
Bug 1254284
Opened 9 years ago
Closed 7 years ago
Remove the dependency on apps status in the Bluetooth code
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: fabrice, Unassigned)
References
Details
Keeping the permission is ok, but everything that is gated by AvailableIn=CertifiedApps needs to be chrome only instead.
http://mxr.mozilla.org/mozilla-central/source/dom/webidl/BluetoothAdapter.webidl#38
Reporter | ||
Updated•9 years ago
|
Summary: Remove the dependency on apps status in the NFC code → Remove the dependency on apps status in the Bluetooth code
Updated•9 years ago
|
Component: General → Bluetooth
Comment 1•9 years ago
|
||
A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/116992817
Comment 2•9 years ago
|
||
Hi.
I was looking for how to contibute and Mr. Fabrice linked me to this bug.
This is my first bug so Im not familiarized with the process.
Fabrice told me that this task is similar to this other -> https://bugzilla.mozilla.org/show_bug.cgi?id=1254282 but Im not fully sure about what should I do.
Any help?
Regards!
Comment 3•9 years ago
|
||
Hi,
(In reply to Penserbjorne from comment #2)
> This is my first bug so Im not familiarized with the process.
Please check link below on how to submit a patch and refer to bug 1254282 as an example.
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch
> Fabrice told me that this task is similar to this other ->
> https://bugzilla.mozilla.org/show_bug.cgi?id=1254282 but Im not fully sure
> about what should I do.
I'm unsure about the full picture either, but based on comment 1 I think you can replace |AvailableIn=CertifiedApps| with |ChromeOnly| in the webidl [1] first. You may have to modify correspondent implementation [2][3] accordingly.
[1] http://mxr.mozilla.org/mozilla-central/source/dom/webidl/BluetoothAdapter.webidl#38
[2] https://dxr.mozilla.org/mozilla-central/source/dom/bluetooth/common/webapi/BluetoothAdapter.h
[3] https://dxr.mozilla.org/mozilla-central/source/dom/bluetooth/common/webapi/BluetoothAdapter.cpp
Comment 4•7 years ago
|
||
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•