Closed Bug 1088527 Opened 8 years ago Closed 8 years ago

BluetoothOppManager.IsConnected() returns false even though OPP profile has connected.

Categories

(Firefox OS Graveyard :: Bluetooth, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
2.2 S3 (9jan)

People

(Reporter: jaliu, Assigned: jaliu)

References

Details

Attachments

(1 file, 1 obsolete file)

BluetoothOppManager.IsConnected() couldn't represent the connection status
of Opp profile since it only return true if there is an ongoing file-transfer session.

This function was used to named |IsTransferring()| in Bug 827267, however,
it was renamed to |IsConnected()| when Bug 842948 landed.

Modified |IsConnected()| to let it indicate whether profile connected or not.
Assignee: nobody → jaliu
Status: NEW → ASSIGNED
Blocks: 1066461
Attachment #8510876 - Flags: review?(btian)
Comment on attachment 8510876 [details] [diff] [review]
Modify BluetoothOppManager.IsConnected() to indicate whether profile is connected or not. (v1)

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

r=me. Please 1) modify BluetoothOppManager under bluez and bluetooth2 as well (total 4 files) and 2) ensure the patch is landed together with bug 1066461 gaia change.
Attachment #8510876 - Flags: review?(btian) → review+
The patch of Bug 1066461 has been landed to m-c.

I've put #Attachment #8510876 [details] [diff] on try server.
https://tbpl.mozilla.org/?tree=Try&rev=6c0c240d310f
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/288ea8616ac1
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S3 (9jan)
Blocks: 1118644
You need to log in before you can comment on or make changes to this bug.