1163914 - Expose connected external display list to gaia system app

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: General, defect)

Description

[Luke Chang [:lchang] (inactive)]

To avoid inventing a new API in this proof-of-concept stage, we propose using mozContentEvent/mozChromeEvent instead. Please refer to the Wiki [1] for our proposal.

[1] https://wiki.mozilla.org/Firefox_OS/1UA_Multi-Screen#Gaia

Comment 1

[Luke Chang [:lchang] (inactive)]

Attachment: WIP Patch

Attach my WIP patch here.

Comment 2

[Luke Chang [:lchang] (inactive)]

Attachment: WIP patch

Change format of the attachment.

Comment 3

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

We have similar functionality requirement for Presentation API device management and we've already create an API for it [1]. I think we can plug the external display to it without using mozChromeEvent.

[1] https://dxr.mozilla.org/mozilla-central/source/dom/webidl/PresentationDeviceInfoManager.webidl#17

Comment 4

[Luke Chang [:lchang] (inactive)]

Yes, we'll try merging multi-screen scenario into Presentation API eventually. For now, however, details aren't clear to us yet how to make it happen so we use mozChromeEvent temporarily until the spec is well-defined. What do you think?

Comment 5

[Luke Chang [:lchang] (inactive)]

Attachment: WIP patch

Update my patch.

Comment 6

[Shelly Lin [:shelly]]

(In reply to Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz) from comment #3)
> We have similar functionality requirement for Presentation API device
> management and we've already create an API for it [1]. I think we can plug
> the external display to it without using mozChromeEvent.
> 
> [1]
> https://dxr.mozilla.org/mozilla-central/source/dom/webidl/
> PresentationDeviceInfoManager.webidl#17

Yes, integrate with Presentation API is one of our goals, but as we have discussed earlier, 1UA-multiscreen and Presentation API are far from integration, so we plan to stick on our current design (even though they are just for experiment) for the first phase. We will sure integrate with Presentation API in the future, thanks for pointing that out.

Comment 7

[Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz)]

There are two individual parts for integrating into Presentation API, one for launching display and another for discovering display. The PresentationDeviceInfoManager is the later and is landed on master already. This API already implement an event passing mechanism, so you can write less code to fulfill your requirement. You can get this by hooking up the device list to PresentationDeviceManager in Bug 1133641, then you can avoid those shell.js hack. :)

Comment 8

[Shelly Lin [:shelly]]

I've filed a task of integrating with PresentationDeviceInfoManager of Presentation API (Bug 1173263). Once it is done, System app no longer need to maintain a list of connected devices.

Comment 9

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

After discussion with the team (Evelyn, Tommy, Luke), it's not really necessary to have this for flip phone's use case.

Comment 10

[Chih-Hsuan Kuo [:tommykuo]]

Attachment: Bug-1163914-hg.patch

This patch will not land to m-c. This is a temporal method to demo multi-screen feature. I'll integrate the display list into device manager of Presentation API.

Comment 11

[Chih-Hsuan Kuo [:tommykuo]]

Attachment: Bug-1163914-hg.patch

Comment 12

[Chih-Hsuan Kuo [:tommykuo]]

Due to we'll integrate multi-screen feature into Presentation API. So, this bug can be closed. See Bug 1161440.