| commit | 564085f7fe6d3a93da77bf6de29d0d514a5b9e66 | |
| author | dalecurtis <dalecurtis@chromium.org> | |
| Wed, 17 Dec 2014 00:33:47 +0000 (16 16:33 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 17 Dec 2014 00:34:14 +0000 (17 00:34 +0000) | ||
| tree | c162bdfe88bdf8d8756dab18f4a7d1f4210897e1 | treesnapshot (tar.gz zip) |
| parent | 42466460ec1a5c0f2953d13e57d0d290cb38ca21 | commitdiff |
Fix broken config change handling in the mojo demuxers.
The null buffers were not handled correctly. The config change
queue system was also incorrect. OnBufferReady() may be in the
process of handling a Read() call, so the read_cb_ can't be reset
immediately once the OnXXXConfigChanged() message is received, it
must wait for the kConfigChange status buffer.
BUG=none
TEST=pipeline integration tests \o/
Review URL: https://codereview.chromium.org/791983004
Cr-Commit-Position: refs/heads/master@{#308703}
The null buffers were not handled correctly. The config change
queue system was also incorrect. OnBufferReady() may be in the
process of handling a Read() call, so the read_cb_ can't be reset
immediately once the OnXXXConfigChanged() message is received, it
must wait for the kConfigChange status buffer.
BUG=none
TEST=pipeline integration tests \o/
Review URL: https://codereview.chromium.org/791983004
Cr-Commit-Position: refs/heads/master@{#308703}