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 #5 id:120001 of https://codereview.chromium.org/2437863004/ )

Reason for revert:
Breaks chromium.webrtc bots due to excessive logging.

https://build.chromium.org/p/chromium.webrtc/builders/Win7%20Tester/builds/30786
https://build.chromium.org/p/chromium.webrtc/builders/Win8%20Tester/builds/29755
https://build.chromium.org/p/chromium.webrtc/builders/Win10%20Tester/builds/11600

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
> Cr-Commit-Position: refs/heads/master@{#427996}

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
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=619533
Committed: https://crrev.com/820c7d29fa971b7ab86f5cf554e6a3b2c547c21b
Cr-Commit-Position: refs/heads/master@{#428015}

[sakal-chromium, 2016-10-27 14:08:21]

The CQ bit was checked by sakal@chromium.org

[sakal-chromium, 2016-10-27 14:08:22]

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

[sakal-chromium, 2016-10-27 14:08:45]

Breakage is because of excessive logging:
[2932:5868:1027/055220:ERROR:track_audio_renderer.cc(59)] Pulling audio out of
shifter to be played 55 ms from now. 512
[2932:5868:1027/055220:ERROR:track_audio_renderer.cc(59)] Pulling audio out of
shifter to be played 57 ms from now. 512
[5792:5848:1027/055220:ERROR:track_audio_renderer.cc(59)] Pulling audio out of
shifter to be played 47 ms from now. 512

Excessive logging can be reproduced locally by running:
./out/Debug/browser_tests
--gtest_filter="WebrtcEventLogApiTest.TestStartStopWebRtcEventLogging" --verbose

[commit-bot: I haz the power, 2016-10-27 14:08:46]

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-10-27 14:08:47]

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

[commit-bot: I haz the power, 2016-10-27 14:08:48]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[tommi (sloooow) - chröme, 2016-10-27 14:10:01]

The CQ bit was checked by tommi@chromium.org

[tommi (sloooow) - chröme, 2016-10-27 14:10:02]

lgtm

[commit-bot: I haz the power, 2016-10-27 14:10:17]

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-10-27 14:11:48]

Description was changed from

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

Reason for revert:
Breaks chromium.webrtc bots due to excessive logging.

https://build.chromium.org/p/chromium.webrtc/builders/Win7%20Tester/builds/30786
https://build.chromium.org/p/chromium.webrtc/builders/Win8%20Tester/builds/29755
https://build.chromium.org/p/chromium.webrtc/builders/Win10%20Tester/builds/1...

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
> Cr-Commit-Position: refs/heads/master@{#427996}

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

to

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

Reason for revert:
Breaks chromium.webrtc bots due to excessive logging.

https://build.chromium.org/p/chromium.webrtc/builders/Win7%20Tester/builds/30786
https://build.chromium.org/p/chromium.webrtc/builders/Win8%20Tester/builds/29755
https://build.chromium.org/p/chromium.webrtc/builders/Win10%20Tester/builds/1...

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
> Cr-Commit-Position: refs/heads/master@{#427996}

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

[commit-bot: I haz the power, 2016-10-27 14:11:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-27 14:13:26]

Description was changed from

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

Reason for revert:
Breaks chromium.webrtc bots due to excessive logging.

https://build.chromium.org/p/chromium.webrtc/builders/Win7%20Tester/builds/30786
https://build.chromium.org/p/chromium.webrtc/builders/Win8%20Tester/builds/29755
https://build.chromium.org/p/chromium.webrtc/builders/Win10%20Tester/builds/1...

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
> Cr-Commit-Position: refs/heads/master@{#427996}

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

to

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

Reason for revert:
Breaks chromium.webrtc bots due to excessive logging.

https://build.chromium.org/p/chromium.webrtc/builders/Win7%20Tester/builds/30786
https://build.chromium.org/p/chromium.webrtc/builders/Win8%20Tester/builds/29755
https://build.chromium.org/p/chromium.webrtc/builders/Win10%20Tester/builds/1...

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
> Cr-Commit-Position: refs/heads/master@{#427996}

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

Committed: https://crrev.com/820c7d29fa971b7ab86f5cf554e6a3b2c547c21b
Cr-Commit-Position: refs/heads/master@{#428015}
==========

[commit-bot: I haz the power, 2016-10-27 14:13:27]

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