Closed Bug 781692 Opened 12 years ago Closed 12 years ago

[b2g-bluetooth] Make BluetoothAdapter property setting functions always fire a DOMRequest

Categories

(Core :: DOM: Device Interfaces, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17
blocking-basecamp -

People

(Reporter: qdot, Unassigned)

References

Details

Attachments

(1 file, 1 obsolete file)

In BluetoothAdapter, if a Set* call is made and the member being set is already equal to the requested value, the function will return NS_OK instead of filling in and returning a DOMRequest. For sake of consistency, all these functions should always fire DOM requests.
For blocking-basecamp: This needs to be fixed so we don't have to check edge cases in gaia. Should be a fairly quick fix.
blocking-basecamp: --- → ?
Comment on attachment 650726 [details] [diff] [review] v1: Actually fire DOMRequests for Set* functions even when value is the same in BluetoothAdapter Looks good to me. Since we're going to remove GetProperties(), class GetPropertiesTask will not be called anywhere. Please check if it can be removed as well.
Attachment #650726 - Flags: review?(echou) → review+
blocking-basecamp: ? → +
I actually think this is a nice-to-have unless we find that there is actual gaia functionality breaking over this. That said, feel free to land this anyway.
blocking-basecamp: + → -
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: