Turn MediaLog usage from plain wrong into questionable

Turn MediaLog usage from plain wrong into questionable

ReadConcatentatedBoxes() was handing out BoxReader instances that
referred to MediaLog objects that had gone out of scope.

BUG=712310
Review-Url: https://codereview.chromium.org/2966493005
Cr-Commit-Position: refs/heads/master@{#483840}
Committed: https://chromium.googlesource.com/chromium/src/+/777668278d3487a6c1cabde6bea0991b95b5007d

[wdzierzanowski, 2017-06-30 07:49:07]

wdzierzanowski@opera.com changed reviewers:
+ dalecurtis@chromium.org

[wdzierzanowski, 2017-06-30 07:49:08]

Going for the lesser evil here, as I don't want to rewrite too much of
media/cdm/cenc_utils.cc ATM.

PTAL.

[wdzierzanowski, 2017-06-30 07:49:43]

Description was changed from

==========
DNA-61606: Turn MediaLog usage from plain wrong into questionable

ReadConcatentatedBoxes() was handing out BoxReader instances that
referred to MediaLog objects that had gone out of scope.

BUG=712310
==========

to

==========
Turn MediaLog usage from plain wrong into questionable

ReadConcatentatedBoxes() was handing out BoxReader instances that
referred to MediaLog objects that had gone out of scope.

BUG=712310
==========

[DaleCurtis, 2017-06-30 17:50:15]

Whoops, thanks. Agreed with approach. lgtm

[wdzierzanowski, 2017-06-30 19:45:15]

The CQ bit was checked by wdzierzanowski@opera.com

[commit-bot: I haz the power, 2017-06-30 19:45:47]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-30 22:13:46]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498851915279060, "parent_rev":
"6f8af6752bb31acff2c55d531688e65fa93e8917", "commit_rev":
"777668278d3487a6c1cabde6bea0991b95b5007d"}

[commit-bot: I haz the power, 2017-06-30 22:14:01]

Description was changed from

==========
Turn MediaLog usage from plain wrong into questionable

ReadConcatentatedBoxes() was handing out BoxReader instances that
referred to MediaLog objects that had gone out of scope.

BUG=712310
==========

to

==========
Turn MediaLog usage from plain wrong into questionable

ReadConcatentatedBoxes() was handing out BoxReader instances that
referred to MediaLog objects that had gone out of scope.

BUG=712310

Review-Url: https://codereview.chromium.org/2966493005
Cr-Commit-Position: refs/heads/master@{#483840}
Committed:
https://chromium.googlesource.com/chromium/src/+/777668278d3487a6c1cabde6bea0...
==========

[commit-bot: I haz the power, 2017-06-30 22:14:02]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/777668278d3487a6c1cabde6bea0...