[Download Service] Notify Clients of initialization failure |
||
Issue descriptionAdd OnServiceUnavailable() to the Client API. If the service fails the Clients need to be known so they can figure out what to do instead of waiting forever or polling for the service.
,
Jul 17 2017
,
Nov 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/29cb248850060bd37f4105bcd61be7c18c04d599 commit 29cb248850060bd37f4105bcd61be7c18c04d599 Author: Patrik Höglund <phoglund@chromium.org> Date: Tue Nov 07 13:32:12 2017 Rewriting wait-for-video functions to use promises. The end goal here is to get rid of addExpectedEvent() / eventOccurred() and replace with promises, which is more standard. I hope we will end up getting rid of all the callback complexity and replacing with straightforward chains of promises. This also harmonizes us with the new browser tests from kerl@, which uses promises. The next CL will try to get rid of the waitFor functions. Further CLs will eliminate other use of the addExpectedEvent etc functions. Bug: 735347 Change-Id: Ibc6dd5ec15aed21fa7f52b7fcc83f5f4d12fe134 Reviewed-on: https://chromium-review.googlesource.com/747626 Reviewed-by: Guido Urdaneta <guidou@chromium.org> Commit-Queue: Patrik Höglund <phoglund@chromium.org> Cr-Commit-Position: refs/heads/master@{#514458} [modify] https://crrev.com/29cb248850060bd37f4105bcd61be7c18c04d599/content/test/data/media/getusermedia-depth-capture.html [modify] https://crrev.com/29cb248850060bd37f4105bcd61be7c18c04d599/content/test/data/media/getusermedia-real-webcam.html [modify] https://crrev.com/29cb248850060bd37f4105bcd61be7c18c04d599/content/test/data/media/getusermedia.html [modify] https://crrev.com/29cb248850060bd37f4105bcd61be7c18c04d599/content/test/data/media/peerconnection-call.html [modify] https://crrev.com/29cb248850060bd37f4105bcd61be7c18c04d599/content/test/data/media/video_capture_test.html [modify] https://crrev.com/29cb248850060bd37f4105bcd61be7c18c04d599/content/test/data/media/webrtc_test_utilities.js |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Jul 13 2017