New issue
Advanced search Search tips

Issue 644357 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 629707



Sign in to add a comment

mash dbus: SMS support (SMSClient)

Project Member Reported by jamescook@chromium.org, Sep 6 2016

Issue description

SMS in chrome browser is handled over D-Bus. go/dbus-in-mustash

See SMSClient / GsmSMSClient / ModemMessagingClient

Some devices with built-in cellular require receiving an SMS message to activate the service.
Wrapper class NetworkSmsHandler. Used by ash in TraySMS.

Per benchan: We only show received SMS via notifications. The current implementation directly hooks into the DBus interfaces exposed by cromo / ModemManager, and not via shill. So the functionality is pretty much detached from the rest of the networking related bindings.

Plan: Factor out SMS handling from the rest of network handling, then use the client library in ash alone.

 
We are actually planning to migrate SMS to use the Message Center instead of the System Tray which would also solve the problem without having to extract the SMS code from NetworkHandler. (Extracting it would be fine, but we may as well avoid it if unnecessary).

See issue 630641

Labels: Proj-Mustash
Components: Internals>MUS
Components: -Internals>MUS Internals>Services>WindowService
Components: -MUS
Components: -Internals>Services>WindowService Internals>Services>Ash
Labels: -Proj-Mustash-Mash
Labels: -Proj-Mustash Proj-Mash-MultiProcess
Owner: steve...@chromium.org
Status: Assigned (was: Untriaged)
Steven, more dbus bugs. Is this still relevant?

Sign in to add a comment