From 8b8f038e79b1fc32976bdb8a7823318de0947904 Mon Sep 17 00:00:00 2001 From: Shelley Vohr Date: Tue, 24 Mar 2020 10:10:22 -0700 Subject: [PATCH] Add 2020-03-23 wg-api meeting notes (#257) --- wg-api/meeting-notes/2020-03-23.md | 73 ++++++++++++++++++++++++++++++ 1 file changed, 73 insertions(+) create mode 100644 wg-api/meeting-notes/2020-03-23.md diff --git a/wg-api/meeting-notes/2020-03-23.md b/wg-api/meeting-notes/2020-03-23.md new file mode 100644 index 0000000..3d02da1 --- /dev/null +++ b/wg-api/meeting-notes/2020-03-23.md @@ -0,0 +1,73 @@ +# API Working Group + +## Date 3/23/19 + +### Attendees + +* @codebytere +* @nornagon +* @MarshallOfSound +* @itsananderson +* @VerteDinde +* @loc +* @jkleinsc + +### Followup + +- **DONE** @codebytere to de-deprecate existing properties deprecations +- **DONE** @zcbenz to update best practices API doc + +### Agenda + +- Chair rotation + - postponed until next meeting +- https://github.com/electron/electron/pull/20006 + - Closed due to inactivity. + - If there's renewed interest we could reconsider. + - Generally we're trying to avoid breaking APIs without v good reason +- [feat: add APIs for accessing NSUserDefaults in the given domain](https://github.com/electron/electron/pull/22193) + - **Verdict**: We will NOT accept this PR. + - Not accessible from MAS + - Not possible to subscribe to changes, which might encourage polling + - Can be implemented straightforwardly as a native module. +- https://github.com/electron/electron/pull/22701 + - **Verdict:** We want this feature, pending details of the API. + - Use cases include support calls & automated testing + - Should this be exposed on desktopCapturer or somewhere else? + - `WebContents.getStream`..? + - Then you could send it to a different renderer + - @loc and @MarshallOfSound to review in detail +- https://github.com/electron/electron/pull/22774 + - **Verdict:** This needs more experimentation to determine if a better interface is possible. + - `contextBridge` simple objects/functions + - Costly to send large objects over the bridge + - e.g. redux state + - on the order of 10ms + - V8 serializer is much faster for serializable objects + - Going to try some benchmarks to see if we can work around it without an API change. +- https://github.com/electron/governance/pull/232 + - **Verdict:** Approved given backwards compatibility + - This a breaking change + - It could be redone to be backwards-compatible + - Reduce inconsistency + - Let's not break old code if we can avoid it! +- https://github.com/electron/governance/pull/254 + - **Verdict:** Agreed to experiment. + - Exposes `//ui/views` + - Use cases: + - splash screen + - placeholder background + - simple "utility" windows e.g. screensharing + - switch between multiple WebContentsViews (e.g. tabs) + - This should be marked EXPERIMENTAL + - WebContentsView should take WebPreferences rather than taking a WebContents + - What happens if you add a View to multiple parents? + - Could these use cases be addressed by adding a backgroundImage option to BrowserWindow? + - Yes, but we're hoping to see some more imaginative real-world use cases + +## Action Items + +- @MarshallOfSound to clarify on what experimental means in the docs +- @codebytere to research Chromium's Intent to Implement / Intent to Experiment protocols +- @zcbenz to make some revisions to the Views spec PR +- @loc and @MarshallOfSound to review "get tab media stream" PR