Pass buffer offset to PlayOutputBuffer() call

Pass buffer offset to PlayOutputBuffer() call

DequeueOutputBuffer returns a buffer index along with size and offset information.
The offset is however ignored in PlayOutputBuffer() call.
This change fixes this issue by passing the offset to PlayOutputBuffer().

BUG=519058
Committed: https://crrev.com/5ffd27c488642c43479a2d9d1450ce3057c20c3c
Cr-Commit-Position: refs/heads/master@{#344924}

[qinmin, 2015-08-10 23:27:57]

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

[qinmin, 2015-08-10 23:27:57]

PTAL

[Tima Vaisburd, 2015-08-11 19:25:40]

lgtm % nit

https://codereview.chromium.org/1285733002/diff/1/media/base/android/media_co...
File media/base/android/media_codec_bridge.cc (right):

https://codereview.chromium.org/1285733002/diff/1/media/base/android/media_co...
media/base/android/media_codec_bridge.cc:708: uint8* buffer =
nit: This code is almost the same as in CopyFromOutputBuffer(). Do you think it
makes sense to have a helper function? If not, I would at least make the code
look identical: same variable names, same CHECK_LE.

[qinmin, 2015-08-20 18:34:21]

https://codereview.chromium.org/1285733002/diff/1/media/base/android/media_co...
File media/base/android/media_codec_bridge.cc (right):

https://codereview.chromium.org/1285733002/diff/1/media/base/android/media_co...
media/base/android/media_codec_bridge.cc:708: uint8* buffer =
On 2015/08/11 19:25:40, Tima Vaisburd wrote:
> nit: This code is almost the same as in CopyFromOutputBuffer(). Do you think
it
> makes sense to have a helper function? If not, I would at least make the code
> look identical: same variable names, same CHECK_LE.

Done.

[qinmin, 2015-08-20 18:34:31]

The CQ bit was checked by qinmin@chromium.org

[qinmin, 2015-08-20 18:34:32]

The patchset sent to the CQ was uploaded after l-g-t-m from timav@chromium.org
Link to the patchset: https://codereview.chromium.org/1285733002/#ps20001
(title: "add a separate function to get output buffer address")

[commit-bot: I haz the power, 2015-08-20 18:36:23]

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

[commit-bot: I haz the power, 2015-08-21 00:41:08]

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

[commit-bot: I haz the power, 2015-08-21 00:41:09]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[qinmin, 2015-08-21 16:59:15]

Patchset #2 (id:20001) has been deleted

[qinmin, 2015-08-21 16:59:18]

The CQ bit was checked by qinmin@chromium.org

[qinmin, 2015-08-21 16:59:18]

The patchset sent to the CQ was uploaded after l-g-t-m from timav@chromium.org
Link to the patchset: https://codereview.chromium.org/1285733002/#ps40001
(title: "add a separate function to get output buffer address")

[commit-bot: I haz the power, 2015-08-21 16:59:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1285733002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1285733002/40001

[commit-bot: I haz the power, 2015-08-21 20:16:54]

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

[commit-bot: I haz the power, 2015-08-21 20:16:55]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[qinmin, 2015-08-21 20:46:03]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2015-08-21 20:47:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1285733002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1285733002/40001

[qinmin, 2015-08-21 23:04:27]

The CQ bit was checked by qinmin@chromium.org

[qinmin, 2015-08-21 23:04:27]

The patchset sent to the CQ was uploaded after l-g-t-m from timav@chromium.org
Link to the patchset: https://codereview.chromium.org/1285733002/#ps60001
(title: "fix bot")

[commit-bot: I haz the power, 2015-08-21 23:05:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1285733002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1285733002/60001

[commit-bot: I haz the power, 2015-08-21 23:39:44]

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

[commit-bot: I haz the power, 2015-08-21 23:39:45]

Try jobs failed on following builders:
  android_compile_dbg on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_compile...)

[Tima Vaisburd, 2015-08-21 23:44:37]

The CQ bit was checked by timav@chromium.org

[commit-bot: I haz the power, 2015-08-21 23:45:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1285733002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1285733002/60001

[commit-bot: I haz the power, 2015-08-22 00:04:24]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-08-22 00:04:53]

Patchset 3 (id:??) landed as
https://crrev.com/5ffd27c488642c43479a2d9d1450ce3057c20c3c
Cr-Commit-Position: refs/heads/master@{#344924}