|
|
Chromium Code Reviews
DescriptionCheck ExecutionContext before calling isContextDestroyed()
BUG=677878
TEST=mediastream layouttests pass
Committed: https://crrev.com/f6a0f4b7f8abe662390f61e1d5133a0894bb6080
Cr-Commit-Position: refs/heads/master@{#441595}
Patch Set 1 #
Messages
Total messages: 17 (10 generated)
The CQ bit was checked by jrummell@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
jrummell@chromium.org changed reviewers: + tommi@chromium.org
PTAL. See bug for crash details.
LGTM
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by tommi@chromium.org
lgtm
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch.
Bot data: {"patchset_id": 1, "attempt_start_ts": 1483596304258210, "parent_rev":
"cc53ddbb49d378d18003915a0f2cded1a4dfea31", "commit_rev":
"620efd4a8a3a520e9e3941209a759759727f0165"}
Message was sent while issue was closed.
Description was changed from ========== Check ExecutionContext before calling isContextDestroyed() BUG=677878 TEST=mediastream layouttests pass ========== to ========== Check ExecutionContext before calling isContextDestroyed() BUG=677878 TEST=mediastream layouttests pass Review-Url: https://codereview.chromium.org/2610233002 ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== Check ExecutionContext before calling isContextDestroyed() BUG=677878 TEST=mediastream layouttests pass Review-Url: https://codereview.chromium.org/2610233002 ========== to ========== Check ExecutionContext before calling isContextDestroyed() BUG=677878 TEST=mediastream layouttests pass Committed: https://crrev.com/f6a0f4b7f8abe662390f61e1d5133a0894bb6080 Cr-Commit-Position: refs/heads/master@{#441595} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/f6a0f4b7f8abe662390f61e1d5133a0894bb6080 Cr-Commit-Position: refs/heads/master@{#441595}
Message was sent while issue was closed.
sigbjornf@opera.com changed reviewers: + sigbjornf@opera.com
Message was sent while issue was closed.
Switching MediaStream to be a ContextClient would be a workable alternative. |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
