New issue
Advanced search Search tips

Issue 612328 link

Starred by 10 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Task


Sign in to add a comment

Content Modularization Project: Device service

Project Member Reported by jam@chromium.org, May 16 2016

Issue description

This tracks having a top level device service. This would live in services/device. It would have getters for various actual device interfaces, like gamepad, bluetooth etc. All these would eventually run in a separate process (except probably on Android where system limitations prevent having too many processes).
 

Comment 1 by jam@chromium.org, May 16 2016

A list from looking at content/browser & devices:
battery_status, bluetooth, device_sensors, gamepad, geolocation, nfc, power_save_blocker, serial, screen_orientation, time_zone_monitor, usb, vibration, vr, wake_lock

Comment 2 by jam@chromium.org, May 16 2016

Blockedon: 612322 612319 612284

Comment 3 by jam@chromium.org, May 16 2016

Blockedon: 612330

Comment 4 by jam@chromium.org, May 16 2016

Blockedon: -612284

Comment 5 by jam@chromium.org, May 16 2016

Blockedon: 612334

Comment 6 by jam@chromium.org, May 16 2016

Blockedon: 612337

Comment 7 by jam@chromium.org, May 16 2016

Blockedon: 612339

Comment 8 by jam@chromium.org, May 16 2016

Blockedon: 612341

Comment 9 by jam@chromium.org, May 17 2016

Blockedon: 612342

Comment 10 by jam@chromium.org, May 17 2016

Blockedon: 612346
Owner: blundell@chromium.org
Status: Started (was: Available)
I'm working on this effort. Taking the bug simply to signify that someone is actively working on this from an end-to-end POV, but of course more than happy to coordinate with anyone else who is working or wants to work on it.

Comment 12 by leon....@intel.com, Sep 14 2016

Hi, blundell@, I'd like to take part in this work as your assistant. I've done some mojofication work and I think I'll also be proved helpful here. ;-)
# After reading codes/docs inside //services/, //content/browser/mojo/ and content/common/mojo/, I've got some knowledge about Service/Shell etc. and how they're deployed in Chrome. I understand we want to create a new device Service here.

Maybe I could start with mojofying some device piece still using legacy IPC, or whatever work following your plan. Thanks~
Great! Communicated with you offline to start to determine work breakdown.
Blockedon: 647247

Comment 15 by st...@chromium.org, Sep 15 2016

Cc: st...@chromium.org
Project Member

Comment 16 by bugdroid1@chromium.org, Nov 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/372d47d28a581be1a9baec3505b805d982129e37

commit 372d47d28a581be1a9baec3505b805d982129e37
Author: blundell <blundell@chromium.org>
Date: Tue Nov 01 01:34:30 2016

Introduce Device Service

This CL introduces the Device Service into //services/device along
with the infrastructure required in //content for the renderer to
connect to it and the browser to host it as an embedded service.

The service is currently empty. A followup CL will move the first
device-related service to be host in the Device Service instead of in
//content/browser.

BUG=612328

Review-Url: https://codereview.chromium.org/2451413003
Cr-Commit-Position: refs/heads/master@{#428904}

[modify] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/content/browser/BUILD.gn
[modify] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/content/browser/browser_context.cc
[modify] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/content/public/app/BUILD.gn
[add] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/services/device/BUILD.gn
[add] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/services/device/OWNERS
[add] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/services/device/device_service.cc
[add] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/services/device/device_service.h
[add] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/services/device/manifest.json
[add] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/services/device/public/cpp/BUILD.gn
[add] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/services/device/public/cpp/constants.cc
[add] https://crrev.com/372d47d28a581be1a9baec3505b805d982129e37/services/device/public/cpp/constants.h

Blockedon: 678920
Blockedon: 680924
Blockedon: 684422
Labels: DeviceService
Blockedon: 686693
Blockedon: 687125
Blockedon: 689382
Blockedon: 689385
Blockedon: 689402
Cc: r...@chromium.org
Cc: -st...@chromium.org
Blockedon: 699790
Blockedon: 709301
Blockedon: 717377
Cc: mattcary@chromium.org
Blockedon: 728223
Blockedon: 728227
Blockedon: 769630
Blockedon: 769631
Blockedon: 769638
Blockedon: 769646
Blockedon: 769647
Blockedon: -612342
Blockedon: 689437
Components: Internals>Services>Device
Labels: Type-Task
Migrating S13N meta bugs to Type=Task, so that they can be distinguished from technical work.
Components: Internals>Services
Blockedon: 850997

Sign in to add a comment