tree b096f429e6a741d3527b3c03a5310a8d7fc55359
parent 42f8f835e70bb1d03e7572c2a3290053680c51e3
author Will Cassella <cassew@google.com> 1628641517 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1628641517 +0000

Release internal lock before running ChunkDemuxer::open_cb_

(passed in during construction), in what is effectivately a tail-call.
It may be useful to, for example, run |ChunkDemuxer::AddId| within that
callback but if the lock is still held then Chrome will crash. This CL
ensures that the lock is released before running the callback (and adds
a unit test to avoid regressing), which is safe due to the call being a
tail-call.

ChunkDemuxer: :Initialize results in a call to ChunkDemuxer::open_cb_
Change-Id: If4d843bd452dfa19bbd62c788e7b1bce9ba8db58
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3088152
Commit-Queue: Will Cassella <cassew@google.com>
Reviewed-by: Matthew Wolenetz <wolenetz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#910593}