A new API proposal: chrome.system.display

[Weidong Guo]

Hi there,

I have a new API proposal to be reviewed.

https://docs.google.com/a/chromium.org/document/d/1Lnl23AR6r98xz8HR5u-ChmIO5TeZfX6BiLMDyLk6f_k/edit?usp=sharing

Thanks,

Weidong.

[Chris Palmer]

Mustafa, are you a good person to take a look at this from an API security perspective?

[Weidong Guo]

Is there any update on the review of the API proposal?

Thanks,

Weidong.

[Weidong Guo]

+ovanieva@

This API is targeting M65, and we are only 10 working days before feature freeze.

On Mon, Dec 11, 2017 at 2:53 PM, Emily Stark <est...@google.com> wrote:

Eric, could you take a look at this for security, please?

[Emily Stark]

I took a quick look and this should be fine for security, though I did leave a comment in the doc requesting screenshots/mocks of the notifications.

[Steven Bennetts]

lgtm with some comments in the doc.

+xiyuan@

[Weidong Guo]

Hi, I landed the implementation CLs:

https://chromium-review.googlesource.com/c/chromium/src/+/844058

https://chromium-review.googlesource.com/c/chromium/src/+/868491

And the API description is added in the extensions/common/api/system_display.idl

I tried to follow the step 3 in Development Phase here to add documentation in https://developer.chrome.com/apps/system_display#method-setDisplayLayout, but it seems the guidelines are outdated and I could not find html source code for the page. What is the right steps here?

Thanks,

Weidong.

[Steven Bennetts]

When you update the IDL the public documentation will be updated when  change makes it to stable (I think). I also think there may be a way to view the docuumentation for other channels? There is alos a way to preview the documentation, which I also forget.

Devlin should be able to provide better answers.

[Weidong Guo]

+rdevlin.cronin@ for confirmation.

Thanks stevenjb@.