Passing key frame info flag renderer to browser process

Passing key frame info flag renderer to browser process

For MediaSourcePlayer, we can use this flag to reduce the browser seek when codec is recreated.
Will implement the logic to reduce browser seek in a follow up CL.

BUG=304234
Committed: https://crrev.com/6f185c3f517b95fcfbeff5c8361aba5389c4215e
Cr-Commit-Position: refs/heads/master@{#313579}

[qinmin, 2015-01-27 01:22:24]

qinmin@chromium.org changed reviewers:
+ wolenetz@chromium.org

[qinmin, 2015-01-27 01:22:25]

PTAL

[qinmin, 2015-01-28 18:11:25]

qinmin@chromium.org changed reviewers:
+ tsepez@chromium.org

[qinmin, 2015-01-28 18:11:25]

+tsepez for ipc/

[Tom Sepez, 2015-01-28 18:14:07]

Messages LGTM

[wolenetz, 2015-01-28 18:26:18]

lgtm. There's a little inconsistency around the naming of the new field:
"end_of_stream" vs "is_key_frame". It looks like the "is_" is inherited from the
DecoderBuffer naming. I prefer keeping the "is_", since otherwise a maintainer
might assume a field named "key_frame" might contain some reference to the
keyframe this buffer is dependent upon. Do you think we should change
end_of_stream to is_end_of_stream to match?

Also, I've updated the bug to reflect the plan I had to fix it. Please let me
know if you want to take over the bug or if you have another approach in mind.

Thanks!
Matt

[qinmin, 2015-01-28 19:39:55]

renamed end_of_stream to is_end_of_stream

[qinmin, 2015-01-28 19:40:00]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2015-01-28 19:41:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/872253008/20001

[commit-bot: I haz the power, 2015-01-28 20:57:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-28 20:58:43]

Patchset 2 (id:??) landed as
https://crrev.com/6f185c3f517b95fcfbeff5c8361aba5389c4215e
Cr-Commit-Position: refs/heads/master@{#313579}