Add support for RGB frames in MediaCodecVideoEncoder.

Add support for RGB frames in MediaCodecVideoEncoder.

BUG=webrtc:7749
Review-Url: https://codereview.webrtc.org/3002263002
Cr-Commit-Position: refs/heads/master@{#19554}
Committed: https://chromium.googlesource.com/external/webrtc/+/160e32c34f629be1fc7e634a085d9dc70a77b908

[sakal, 2017-08-25 08:44:29]

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

[sakal, 2017-08-25 08:44:29]

PTAL

[sakal, 2017-08-25 08:46:44]

Patchset #2 (id:20001) has been deleted

[magjed_webrtc, 2017-08-25 13:15:43]

lgtm

https://codereview.webrtc.org/3002263002/diff/40001/webrtc/sdk/android/api/or...
File webrtc/sdk/android/api/org/webrtc/MediaCodecVideoEncoder.java (right):

https://codereview.webrtc.org/3002263002/diff/40001/webrtc/sdk/android/api/or...
webrtc/sdk/android/api/org/webrtc/MediaCodecVideoEncoder.java:619:
drawer.drawTexture(textureBuffer, new Matrix(), width, height, 0 /* viewportX
*/,
Comment /* renderMatrix */. Also, we should probably comment in the interface
how this works because it was confusing to me.

[sakal, 2017-08-25 13:28:35]

Description was changed from

==========
Add suport for RGB frames MediaCodecVideoEncoder.

BUG=webrtc:7749
==========

to

==========
Add suport for RGB frames in MediaCodecVideoEncoder.

BUG=webrtc:7749
==========

[sakal, 2017-08-28 08:06:30]

https://codereview.webrtc.org/3002263002/diff/40001/webrtc/sdk/android/api/or...
File webrtc/sdk/android/api/org/webrtc/MediaCodecVideoEncoder.java (right):

https://codereview.webrtc.org/3002263002/diff/40001/webrtc/sdk/android/api/or...
webrtc/sdk/android/api/org/webrtc/MediaCodecVideoEncoder.java:619:
drawer.drawTexture(textureBuffer, new Matrix(), width, height, 0 /* viewportX
*/,
On 2017/08/25 13:15:43, magjed_webrtc wrote:
> Comment /* renderMatrix */. Also, we should probably comment in the interface
> how this works because it was confusing to me.

Done.

[sakal, 2017-08-28 08:06:37]

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

[commit-bot: I haz the power, 2017-08-28 08:06:47]

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-28 08:45:56]

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

[commit-bot: I haz the power, 2017-08-28 08:45:57]

Dry run: This issue passed the CQ dry run.

[sakal, 2017-08-28 09:38:51]

The CQ bit was checked by sakal@webrtc.org

[sakal, 2017-08-28 09:38:51]

The patchset sent to the CQ was uploaded after l-g-t-m from magjed@webrtc.org
Link to the patchset: https://codereview.webrtc.org/3002263002/#ps60001 (title:
"Address comments.")

[commit-bot: I haz the power, 2017-08-28 09:38:57]

CQ is trying da patch.

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

[sakal, 2017-08-28 09:38:59]

The CQ bit was unchecked by sakal@webrtc.org

[sakal, 2017-08-28 09:39:18]

Description was changed from

==========
Add suport for RGB frames in MediaCodecVideoEncoder.

BUG=webrtc:7749
==========

to

==========
Add support for RGB frames in MediaCodecVideoEncoder.

BUG=webrtc:7749
==========

[sakal, 2017-08-28 09:39:23]

The CQ bit was checked by sakal@webrtc.org

[commit-bot: I haz the power, 2017-08-28 09:39:27]

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-28 09:41:39]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1503913163928060,
"parent_rev": "ef423d4298686cda29e3bd74e1152c8e52b97e28", "commit_rev":
"160e32c34f629be1fc7e634a085d9dc70a77b908"}

[commit-bot: I haz the power, 2017-08-28 09:41:51]

Description was changed from

==========
Add support for RGB frames in MediaCodecVideoEncoder.

BUG=webrtc:7749
==========

to

==========
Add support for RGB frames in MediaCodecVideoEncoder.

BUG=webrtc:7749

Review-Url: https://codereview.webrtc.org/3002263002
Cr-Commit-Position: refs/heads/master@{#19554}
Committed:
https://chromium.googlesource.com/external/webrtc/+/160e32c34f629be1fc7e634a0...
==========

[commit-bot: I haz the power, 2017-08-28 09:41:52]

Committed patchset #3 (id:60001) as
https://chromium.googlesource.com/external/webrtc/+/160e32c34f629be1fc7e634a0...