Move 'mdat' advancement logic to make more sense.

Move 'mdat' advancement logic to make more sense.

This defers the change from kEmittingSamples to kParsingBoxes until after the
current 'mdat' atom has been cleared in MP4StreamParser.

BUG=
TEST=MP4StreamParserTest


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148977

[strobe_, 2012-07-28 06:07:39]

Another tiny CL with almost no semantic impact. If these changes are too trivial
to get their own CL in the future, LMK and I'll bundle them. Thanks!

[acolwell GONE FROM CHROMIUM, 2012-07-30 15:45:49]

LGTM % nit

http://codereview.chromium.org/10821081/diff/3001/media/mp4/mp4_stream_parser.cc
File media/mp4/mp4_stream_parser.cc (right):

http://codereview.chromium.org/10821081/diff/3001/media/mp4/mp4_stream_parser...
media/mp4/mp4_stream_parser.cc:323: if (!queue_.Trim(mdat_tail_)) return false;
nit: Put return on its own line.

[strobe_, 2012-07-30 16:33:38]

http://codereview.chromium.org/10821081/diff/3001/media/mp4/mp4_stream_parser.cc
File media/mp4/mp4_stream_parser.cc (right):

http://codereview.chromium.org/10821081/diff/3001/media/mp4/mp4_stream_parser...
media/mp4/mp4_stream_parser.cc:323: if (!queue_.Trim(mdat_tail_)) return false;
On 2012/07/30 15:45:50, acolwell wrote:
> nit: Put return on its own line.

Done.

[commit-bot: I haz the power, 2012-07-30 16:33:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/strobe@google.com/10821081/6002

[commit-bot: I haz the power, 2012-07-30 18:16:59]

Change committed as 148977