| commit | 2e50e6f660c9fb42f3f63c127aae2c66b054341d | |
| author | dalecurtis <dalecurtis@chromium.org> | |
| Fri, 9 Jan 2015 02:42:52 +0000 (8 18:42 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Fri, 9 Jan 2015 02:43:41 +0000 (9 02:43 +0000) | ||
| tree | a67cabc154e65fbfec25d625f745400767925c30 | treesnapshot (tar.gz zip) |
| parent | fa12ca02a4a47b9321b46a1ac64fc57a2220d387 | commitdiff |
Revert current time clamp on ended; clamp paused time instead.
Clamping the current time within Pipeline leads to a race
a race condition while using |renderer_ended_|. The problem
that code was added for can mostly be fixed by instead clamping
the pause time in WebMediaPlayerImpl.
I've verified the HTMLMediaElement cleanup in another CL still
passes all tests: https://codereview.chromium.org/456343002/
BUG=446637
TEST=layout tests, unittests.
Review URL: https://codereview.chromium.org/843863002
Cr-Commit-Position: refs/heads/master@{#310686}
Clamping the current time within Pipeline leads to a race
a race condition while using |renderer_ended_|. The problem
that code was added for can mostly be fixed by instead clamping
the pause time in WebMediaPlayerImpl.
I've verified the HTMLMediaElement cleanup in another CL still
passes all tests: https://codereview.chromium.org/456343002/
BUG=446637
TEST=layout tests, unittests.
Review URL: https://codereview.chromium.org/843863002
Cr-Commit-Position: refs/heads/master@{#310686}