Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2043)

Unified Diff: services/video_capture/public/interfaces/video_capture_device_factory.mojom

Issue 2480203002: ui: Cleanup class/struct forward declarations (Closed)
Patch Set: Sync CL to position 430550 Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: services/video_capture/public/interfaces/video_capture_device_factory.mojom
diff --git a/services/video_capture/public/interfaces/video_capture_device_factory.mojom b/services/video_capture/public/interfaces/video_capture_device_factory.mojom
index 88c64c15788a527d4ffa8df8565d669bd19dd267..01752012226cdb183d43f83111506f992b9b1f57 100644
--- a/services/video_capture/public/interfaces/video_capture_device_factory.mojom
+++ b/services/video_capture/public/interfaces/video_capture_device_factory.mojom
@@ -34,7 +34,14 @@ interface VideoCaptureDeviceFactory {
// Provides exclusive access to the device identified by |device_descriptor|.
// The access is valid until either the message pipe associated with
// |proxy_request| is closed by the client, or a subsequent call to
- // CreateDeviceProxy() is made.
+ // CreateDeviceProxy() is made. When a subsequent call is made while the
+ // |proxy_request| from a previous call has not yet been closed, the service
+ // "revokes" that previous |proxy_request| (by closing the connection) and the
+ // exclusive access goes to the new |proxy_request|.
+ // Note: This design avoids the issue of it not being clear how soon after a
+ // |proxy_request| has been (asynchronously) discarded a subsequent call to
+ // CreateDeviceProxy() would succeed if it were to deny access while still
+ // in use.
CreateDeviceProxy(VideoCaptureDeviceDescriptor device_descriptor,
VideoCaptureDeviceProxy& proxy_request)
=> (DeviceAccessResultCode result_code);
« no previous file with comments | « services/video_capture/mock_device_factory.cc ('k') | services/video_capture/video_capture_device_factory_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698