Remove AudioBuffersState class.

Remove AudioBuffersState class.

The AudioBuffersState object doesn't appear to be necessary anymore.
Most code either completely ignores this information or doesn't actually
care about the difference between pending_bytes and hardware_delay_bytes.
Also usually only one of the 2 fields was actually being used at a time.
This change removes the class and simply uses an int that represent the
total number of delay bytes.

BUG=125685
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290359

[acolwell GONE FROM CHROMIUM, 2014-08-12 20:22:10]

Please take a look. 

I'm intentionally not changing the parameter units like a comment in part of the
code suggests because I wanted to keep this change focused on class elimination
and not muddy it with other behavior changes.

[DaleCurtis, 2014-08-12 20:39:35]

Thanks for doing this! Is your end goal http://crbug.com/125685 ?

[DaleCurtis, 2014-08-12 20:39:56]

Also, lgtm once the bots are happy. I suspect there are some PPAPI / NaCl fixes
you're missing.

[acolwell GONE FROM CHROMIUM, 2014-08-12 22:11:59]

On 2014/08/12 20:39:35, DaleCurtis wrote:
> Thanks for doing this! Is your end goal http://crbug.com/125685 ?

I don't know yet. Bytes is definitely wrong, but I haven't figured out whether
TimeDelta or simply frames is better. I just happened to notice this while doing
some Mojo stuff and figured I'd send out a CL. :)

[acolwell GONE FROM CHROMIUM, 2014-08-13 03:42:49]

The CQ bit was checked by acolwell@chromium.org

[commit-bot: I haz the power, 2014-08-13 03:43:53]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/acolwell@chromium.org/467833002/40001

[commit-bot: I haz the power, 2014-08-13 09:08:43]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-13 09:12:59]

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

[commit-bot: I haz the power, 2014-08-13 09:13:00]

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

[acolwell GONE FROM CHROMIUM, 2014-08-13 16:21:02]

rkc : Please OWNERS review components/copresence changes
palmer: Please OWNERS review content/common/media/audio_messages.h

[rkc, 2014-08-13 16:57:49]

lgtm for copresence/*

[miu, 2014-08-13 21:17:31]

Yeah, I wanted to tackle this to improve the timing information being provided
to the tab capture and Cast/WebRTC pipeline.  I totally agree with this change. 
Also agree with what you noticed, about "bytes" being a really poor choice for
representing the amount of data buffered.  Will resolve this in some later
change (per bug 125685).

[acolwell GONE FROM CHROMIUM, 2014-08-15 16:32:43]

palmer@: ping for OWNERS review. It is a trivial removal of an unused include so
the review should take only a few seconds.

[palmer, 2014-08-18 17:39:14]

Rubberstamp LGTM

[acolwell GONE FROM CHROMIUM, 2014-08-18 18:02:11]

The CQ bit was checked by acolwell@chromium.org

[commit-bot: I haz the power, 2014-08-18 18:03:23]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/acolwell@chromium.org/467833002/60001

[commit-bot: I haz the power, 2014-08-18 19:51:35]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[commit-bot: I haz the power, 2014-08-18 20:49:19]

Committed patchset #4 (60001) as 290359