-
chcunningham authored
See Intent to Deprecate https://groups.google.com/a/chromium.org/d/msg/blink-dev/x54XtrTyOP8/4-5QZlZzDAAJ MSE's resource is considered 'local' (we don't manage the donwload - the app does), so the HTML5 spec text around firing 'stalled' does not apply. See discussion in https://crbug.com/517240 No change to 'stalled' event for 'remote' (src="somefile.webm") resources. This CL also stops delaying the 'load' event for deferred media player Load() execution. Load() may be deferred for an arbitrary number of seconds, so we should not hold up the window's 'load' event. Prior to this change, 'stalled' was serving the unintended roll of unblocking the the 'load' event for deferred load()s that took > 3 seconds. With stall now removed for MSE, this weird behavior was uncovered and remedied. Bug: 517240, 836951 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I97f2bf972c79582d038e1215f5d630042656bd1c Reviewed-on: https://chromium-review.googlesource.com/982564 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Matthew Wolenetz <wolenetz@chromium.org> Cr-Commit-Position: refs/heads/master@{#562371}
707b821e