[mediacapture-screen-share] Use same enum for or all include or exclude enums? (#278)

• From: François Beaufort via GitHub <sysbot+gh@w3.org>
• Date: Fri, 01 Sep 2023 15:00:06 +0000
• To: public-webrtc-logs@w3.org
• Message-ID: <issues.opened-1877587010-1693580403-sysbot+gh@w3.org>

beaufortfrancois has just created a new issue for https://github.com/w3c/mediacapture-screen-share:

== Use same enum for or all include or exclude enums? ==
May I suggest we add a single enum for "include", "exclude" (not web exposed) and use it where needed?

**Before:**

```webidl
enum SelfCapturePreferenceEnum {
  "include",
  "exclude"
};
enum SystemAudioPreferenceEnum {
  "include",
  "exclude"
};
enum SurfaceSwitchingPreferenceEnum {
  "include",
  "exclude"
};
enum MonitorTypeSurfacesEnum {
  "include",
  "exclude"
};
```

**After:**

```webidl
enum DisplayMediaIncludeOrExclude { "include", "exclude" };

typedef DisplayMediaIncludeOrExclude SystemAudioPreferenceEnum;
typedef DisplayMediaIncludeOrExclude SelfCapturePreferenceEnum;
typedef DisplayMediaIncludeOrExclude SurfaceSwitchingPreferenceEnum;
typedef DisplayMediaIncludeOrExclude MonitorTypeSurfacesEnum;
```

Please view or discuss this issue at https://github.com/w3c/mediacapture-screen-share/issues/278 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 1 September 2023 15:00:08 UTC