ObjC: Support non-native frames in encoder

ObjC: Support non-native frames in encoder

Frames might be non-native, i.e. normal I420 frames, and we should
handle that in the encoder.

BUG=webrtc:7785, webrtc:7924
Review-Url: https://codereview.webrtc.org/2992943002
Cr-Commit-Position: refs/heads/master@{#19245}
Committed: https://chromium.googlesource.com/external/webrtc/+/512bee3dee4afd960e378f951570984e171782b3

[magjed_webrtc, 2017-08-03 09:17:08]

Description was changed from

==========
ObjC: Support non-native frames in encoder

Frames might be non-native, i.e. normal I420 frames, and we should
handle that in the encoder.

BUG=webrtc:7785
==========

to

==========
ObjC: Support non-native frames in encoder

Frames might be non-native, i.e. normal I420 frames, and we should
handle that in the encoder.

BUG=webrtc:7785,webrtc:7924
==========

[magjed_webrtc, 2017-08-03 09:17:08]

magjed@webrtc.org changed reviewers:
+ andersc@webrtc.org

[magjed_webrtc, 2017-08-03 09:18:09]

The CQ bit was checked by magjed@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-08-03 09:18:15]

Dry run: CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-08-03 09:43:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-08-03 09:43:46]

Dry run: This issue passed the CQ dry run.

[magjed_webrtc, 2017-08-03 12:34:41]

Anders - please take a look.

[andersc, 2017-08-03 17:02:10]

lgtm

Looks like basically the same solution that is in the "Implement H264 codec in
Objective-C" CL
https://codereview.webrtc.org/2978623002/diff/140001/webrtc/sdk/objc/Framewor...

[magjed_webrtc, 2017-08-04 09:01:45]

On 2017/08/03 17:02:10, andersc wrote:
> lgtm
> 
> Looks like basically the same solution that is in the "Implement H264 codec in
> Objective-C" CL
>
https://codereview.webrtc.org/2978623002/diff/140001/webrtc/sdk/objc/Framewor...

Yes, I based the code on that :)

[magjed_webrtc, 2017-08-04 09:02:23]

The CQ bit was checked by magjed@webrtc.org

[commit-bot: I haz the power, 2017-08-04 09:02:29]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-08-04 09:05:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1501837343886640, "parent_rev":
"3e8016e1d51e6db21a51b7fcf308b8904aee2f66", "commit_rev":
"512bee3dee4afd960e378f951570984e171782b3"}

[commit-bot: I haz the power, 2017-08-04 09:05:48]

Description was changed from

==========
ObjC: Support non-native frames in encoder

Frames might be non-native, i.e. normal I420 frames, and we should
handle that in the encoder.

BUG=webrtc:7785,webrtc:7924
==========

to

==========
ObjC: Support non-native frames in encoder

Frames might be non-native, i.e. normal I420 frames, and we should
handle that in the encoder.

BUG=webrtc:7785,webrtc:7924

Review-Url: https://codereview.webrtc.org/2992943002
Cr-Commit-Position: refs/heads/master@{#19245}
Committed:
https://chromium.googlesource.com/external/webrtc/+/512bee3dee4afd960e378f951...
==========

[commit-bot: I haz the power, 2017-08-04 09:05:49]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/512bee3dee4afd960e378f951...