| commit | 93f4cc6b2d00db4bdf49190c484329f5d3dd066d | |
| author | ajose <ajose@chromium.org> | |
| Wed, 19 Aug 2015 21:23:45 +0000 (19 14:23 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 19 Aug 2015 21:24:30 +0000 (19 21:24 +0000) | ||
| tree | 372836cb80cc0ebce42c7997b3ca631767eddeb5 | treesnapshot (tar.gz zip) |
| parent | 92e3a0f75d28517ff9e8bc47b11666e436762ee5 | commitdiff |
Simplify content::VideoCaptureImpl to support only a single client.
content::VideoCaptureImpl currently can support multiple
media::VideoCapture::EventHandlers, but this complexity is unnecessary.
See ref'd issue for more discussion.
BUG=299394
TEST=If creating a JS stream with getUserMedia() and then duplicating via stream(stream), a new VCSessionId should be created [1], so the 1:1 mapping from VCSessionId to VCImpl will be maintained.
[1] https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/renderer_host/media/video_capture_manager.cc&type=cs&sq=package:chromium&l=222&rcl=1439335533
Review URL: https://codereview.chromium.org/1272323004
Cr-Commit-Position: refs/heads/master@{#344314}
content::VideoCaptureImpl currently can support multiple
media::VideoCapture::EventHandlers, but this complexity is unnecessary.
See ref'd issue for more discussion.
BUG=299394
TEST=If creating a JS stream with getUserMedia() and then duplicating via stream(stream), a new VCSessionId should be created [1], so the 1:1 mapping from VCSessionId to VCImpl will be maintained.
[1] https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/renderer_host/media/video_capture_manager.cc&type=cs&sq=package:chromium&l=222&rcl=1439335533
Review URL: https://codereview.chromium.org/1272323004
Cr-Commit-Position: refs/heads/master@{#344314}