Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement WebBluetooth in Servo #9392

Closed
dati91 opened this issue Jan 21, 2016 · 15 comments
Closed

Implement WebBluetooth in Servo #9392

dati91 opened this issue Jan 21, 2016 · 15 comments
Labels

Comments

@dati91
Copy link
Contributor

@dati91 dati91 commented Jan 21, 2016

The WebBluetooth API ( https://webbluetoothcg.github.io/web-bluetooth/ ) is currently missing,
so I started working on it. It is very experimental, but the chromium folks already working on it as well.
The current implementation status of WebBluetooth can be found here: https://github.com/WebBluetoothCG/web-bluetooth/blob/gh-pages/implementation-status.md
( Servo is not listed yet. )

The first blocker issue is #4282 (missing the Promise)

@Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Jan 21, 2016

I am not certain we should accept code for this API.

@jdm
Copy link
Member

@jdm jdm commented Jan 21, 2016

@Ms2ger Please elaborate.

@jdm jdm added the A-content/dom label Jan 21, 2016
@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Jan 24, 2016

FYI, we have a team at the University of Szeged who are planning to work on this. We've been collaborating with the Firefox Bluetooth team on which spec(s) to implement, how to hook up to the underlying bluetooth libraries, etc.

@dati91
Copy link
Contributor Author

@dati91 dati91 commented Jan 25, 2016

Yes, we will constantly update our progress. And there will be a link to our branch soon.

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Jan 25, 2016

@dati91 Sorry I didn't recognize your username, Attila! I've only talked with @akiss77 and @loki04 to date :-)

@dati91
Copy link
Contributor Author

@dati91 dati91 commented Jan 26, 2016

We will use this repository: https://github.com/szeged/servo

@jasonwilliams
Copy link
Contributor

@jasonwilliams jasonwilliams commented Feb 10, 2016

Should this be marked as assigned then?

@jdm
Copy link
Member

@jdm jdm commented Feb 10, 2016

@Jayflux Generally it's less important to mark issues that aren't in the categories with high-visibility (and/or surfaced by tools like http://servo.github.io/servo-starters/).

@Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Feb 15, 2016

Given that this api is full of promises, I'm somewhat confused how that's going to be dealt with.

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Feb 15, 2016

@Ms2ger This work depends on https://bugzilla.mozilla.org/show_bug.cgi?id=911216 landing and then picking up an smup. We are coordinating with the people working on that :-)

@dati91
Copy link
Contributor Author

@dati91 dati91 commented Mar 18, 2016

A quick update:
The first part of the implementation landed: e7d70cf

The low level bluetooth api code can be found here: https://github.com/servo/devices

Currently working on the middle layer (ipc_channel) communication which will connect the two: https://github.com/szeged/servo/tree/bluetooth-ipc

@dati91
Copy link
Contributor Author

@dati91 dati91 commented Mar 18, 2016

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Mar 23, 2016

Note that #10081 will make this feature not web-visible by default and will require passing a commandline argument to enable it.

@g-ortuno
Copy link

@g-ortuno g-ortuno commented Apr 20, 2016

dati91: I don't think it's necessarily a blocker. Chromium doesn't implement it yet either so we just use an array.

@nox
Copy link
Member

@nox nox commented Sep 30, 2017

NEWS AT 11: WAS DONE MONTHS AGO YAY

@nox nox closed this Sep 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
7 participants
You can’t perform that action at this time.