Revert of Make more media APIs aware of |delay| and |delay_timestamp|

Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset #7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web speech to speak something e.g. speechSynthesis.speak(new SpeechSynthesisUtterance('test'));
The patch also wasn't 'git cl format'ed.
Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
>
> This patch is a part of 'AudioContext.getOutputTimestamp' method implementation [1] and it continues the changes introduced at [2].
>
> The AudioRendererSink::RenderCallback::Render method has same semantics as AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped) -> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) -> AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
>
> BUG=619533
>
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
>
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jameswest@chromium.org,miu@chromium.org,perkj@chromium.org,tommi@chromium.org,mikhail.pozdnyakov@intel.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533
TBR=tommi@chromium.org
NOTRY=true
NOPRESUBMIT=true

Committed: https://crrev.com/2665e952502835c6c37d51153bfa180343635a41
Cr-Commit-Position: refs/heads/master@{#429412}

[David Tseng, 2016-11-02 20:31:34]

The CQ bit was checked by dtseng@chromium.org

[David Tseng, 2016-11-02 20:31:35]

Created Revert of Make more media APIs aware of |delay| and |delay_timestamp|

[commit-bot: I haz the power, 2016-11-02 20:32:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-02 20:44:44]

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

[commit-bot: I haz the power, 2016-11-02 20:44:46]

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

[David Tseng, 2016-11-02 21:21:49]

Description was changed from

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));

Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533
==========

to

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));

Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533
TBR=tommi@chromium.org
NOTRY=true
==========

[David Tseng, 2016-11-02 21:21:54]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-11-02 21:22:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-02 21:40:24]

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

[commit-bot: I haz the power, 2016-11-02 21:40:25]

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

[David Tseng, 2016-11-02 21:51:20]

Description was changed from

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));

Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533
TBR=tommi@chromium.org
NOTRY=true
==========

to

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));
The patch also wasn't 'git cl format'ed.
Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533
TBR=tommi@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

[David Tseng, 2016-11-02 21:51:25]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-11-02 21:51:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[David Tseng, 2016-11-02 21:53:11]

Description was changed from

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));
The patch also wasn't 'git cl format'ed.
Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533
TBR=tommi@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));
The patch also wasn't 'git cl format'ed.
Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533

NOTRY=true
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-11-02 21:53:37]

Description was changed from

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));
The patch also wasn't 'git cl format'ed.
Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533

NOTRY=true
NOPRESUBMIT=true
==========

to

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));
The patch also wasn't 'git cl format'ed.
Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533

NOTRY=true
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-11-02 21:53:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-02 21:56:22]

Description was changed from

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));
The patch also wasn't 'git cl format'ed.
Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
> 
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
> 
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
> 
> BUG=619533
> 
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
> 
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533

NOTRY=true
NOPRESUBMIT=true
==========

to

==========
Revert of Make more media APIs aware of |delay| and |delay_timestamp| (patchset
#7 id:200001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
This breaks built in speech synthesis on Chrome OS. To reproduce, simply use web
speech to speak something e.g. speechSynthesis.speak(new
SpeechSynthesisUtterance('test'));
The patch also wasn't 'git cl format'ed.
Original issue's description:
> Make more media APIs aware of |delay| and |delay_timestamp|
>
> This patch is a part of 'AudioContext.getOutputTimestamp' method
implementation [1] and it continues the changes introduced at [2].
>
> The AudioRendererSink::RenderCallback::Render method has same semantics as
AudioSourceCallback::OnMoreData.
> That caused other changes in the following interfaces:
> - AudioSyncReader::UpdatePendingBytes(uint32_t bytes, uint32_t frames_skipped)
-> AudioSyncReader::PrepareNextData(base::TimeDelta delay, base::TimeTicks
delay_timestamp, uint32_t frames_skipped)
> - AudioDeviceThread::Callback::Process(uint32_t pending_data) ->
AudioDeviceThread::Callback::Process(int64_t pending_data, base::TimeTicks
data_timestamp)
> - AudioDeviceThread is receiving a "packet" of two int64_t values
>
> BUG=619533
>
> [1] https://codereview.chromium.org/2060833002/
> [2] https://codereview.chromium.org/2101303004/
>
> Committed: https://crrev.com/df872a842609926a62e0211868c886d0ec96cff7
> Committed: https://crrev.com/06d9d80a203b9c329fc6d53fde44bae66645a389
> Cr-Original-Commit-Position: refs/heads/master@{#427996}
> Cr-Commit-Position: refs/heads/master@{#428953}

TBR=jochen@chromium.org,chcunningham@chromium.org,jameswest@google.com,jamesw...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=619533
TBR=tommi@chromium.org
NOTRY=true
NOPRESUBMIT=true

Committed: https://crrev.com/2665e952502835c6c37d51153bfa180343635a41
Cr-Commit-Position: refs/heads/master@{#429412}
==========

[commit-bot: I haz the power, 2016-11-02 21:56:23]

Patchset 1 (id:??) landed as
https://crrev.com/2665e952502835c6c37d51153bfa180343635a41
Cr-Commit-Position: refs/heads/master@{#429412}