Revert of Introduce PushPullFIFO class and remove other FIFOs

Revert of Introduce PushPullFIFO class and remove other FIFOs (patchset #15 id:340001 of https://codereview.chromium.org/2549093009/ )

Reason for revert:
https://build.chromium.org/p/chromium.fyi/builders/CrWinClang%20tester/builds/12077

Some death tests for CHECK() are failing on the official build.

Original issue's description:
> This CL is to reduce the complexity of current FIFO structure. 
> 
> 1. The simplified PushPullFIFO replaces old FIFOs into one, and removes the 
> cyclic call structure between AudioDestination and AudioPullFIFO.
> 2. This refactoring will be helpful when the new WebThread for AudioWorklet
> is implemented. (push and pull operation now can be separated.)
> 
> BUG=678678
> 
> Review-Url: https://codereview.chromium.org/2549093009
> Cr-Commit-Position: refs/heads/master@{#448784}
> Committed: https://chromium.googlesource.com/chromium/src/+/41a5385ea755ea3550ce907716c329b559186a2e

TBR=rtoy@chromium.org,olka@chromium.org,haraken@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=678678

[hongchan, 2017-02-08 20:13:07]

The CQ bit was checked by hongchan@chromium.org

[hongchan, 2017-02-08 20:13:07]

Created Revert of Introduce PushPullFIFO class and remove other FIFOs

[commit-bot: I haz the power, 2017-02-08 20:13:31]

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

[hongchan, 2017-02-08 20:14:05]

FYI.

https://build.chromium.org/p/chromium.fyi/builders/CrWinClang%20tester/builds...

[commit-bot: I haz the power, 2017-02-08 20:14:10]

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

[commit-bot: I haz the power, 2017-02-08 20:14:16]

Failed to apply patch for third_party/WebKit/Source/platform/BUILD.gn:
While running git apply --index -p1;
  error: patch failed: third_party/WebKit/Source/platform/BUILD.gn:1699
  error: third_party/WebKit/Source/platform/BUILD.gn: patch does not apply

Patch:       third_party/WebKit/Source/platform/BUILD.gn
Index: third_party/WebKit/Source/platform/BUILD.gn
diff --git a/third_party/WebKit/Source/platform/BUILD.gn
b/third_party/WebKit/Source/platform/BUILD.gn
index
38dbfa54e5dcac26e766c8e4b686f01d89af2bf6..6176634d8b1ad12d2932bb012a1cfa18259b78b3
100644
--- a/third_party/WebKit/Source/platform/BUILD.gn
+++ b/third_party/WebKit/Source/platform/BUILD.gn
@@ -405,10 +405,14 @@
     "audio/AudioDestination.cpp",
     "audio/AudioDestination.h",
     "audio/AudioDestinationConsumer.h",
+    "audio/AudioFIFO.cpp",
+    "audio/AudioFIFO.h",
     "audio/AudioFileReader.h",
     "audio/AudioIOCallback.h",
     "audio/AudioProcessor.cpp",
     "audio/AudioProcessor.h",
+    "audio/AudioPullFIFO.cpp",
+    "audio/AudioPullFIFO.h",
     "audio/AudioResampler.cpp",
     "audio/AudioResampler.h",
     "audio/AudioResamplerKernel.cpp",
@@ -455,8 +459,6 @@
     "audio/MultiChannelResampler.h",
     "audio/Panner.cpp",
     "audio/Panner.h",
-    "audio/PushPullFIFO.cpp",
-    "audio/PushPullFIFO.h",
     "audio/Reverb.cpp",
     "audio/Reverb.h",
     "audio/ReverbAccumulationBuffer.cpp",
@@ -1699,7 +1701,6 @@
     "animation/CompositorAnimationTimelineTest.cpp",
     "animation/CompositorFloatAnimationCurveTest.cpp",
     "animation/TimingFunctionTest.cpp",
-    "audio/PushPullFIFOTest.cpp",
     "blob/BlobDataTest.cpp",
     "clipboard/ClipboardUtilitiesTest.cpp",
     "exported/FilePathConversionTest.cpp",

[Nico, 2017-02-08 20:22:03]

manually created a revert at https://codereview.chromium.org/2686893002/

[hongchan, 2017-02-14 22:46:46]

Description was changed from

==========
Revert of Introduce PushPullFIFO class and remove other FIFOs (patchset #15
id:340001 of https://codereview.chromium.org/2549093009/ )

Reason for revert:
https://build.chromium.org/p/chromium.fyi/builders/CrWinClang%20tester/builds...

Some death tests for CHECK() are failing on the official build.

Original issue's description:
> This CL is to reduce the complexity of current FIFO structure. 
> 
> 1. The simplified PushPullFIFO replaces old FIFOs into one, and removes the 
> cyclic call structure between AudioDestination and AudioPullFIFO.
> 2. This refactoring will be helpful when the new WebThread for AudioWorklet
> is implemented. (push and pull operation now can be separated.)
> 
> BUG=678678
> 
> Review-Url: https://codereview.chromium.org/2549093009
> Cr-Commit-Position: refs/heads/master@{#448784}
> Committed:
https://chromium.googlesource.com/chromium/src/+/41a5385ea755ea3550ce907716c3...

TBR=rtoy@chromium.org,olka@chromium.org,haraken@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=678678
==========

to

==========
Revert of Introduce PushPullFIFO class and remove other FIFOs (patchset #15
id:340001 of https://codereview.chromium.org/2549093009/ )

Reason for revert:
https://build.chromium.org/p/chromium.fyi/builders/CrWinClang%20tester/builds...

Some death tests for CHECK() are failing on the official build.

Original issue's description:
> This CL is to reduce the complexity of current FIFO structure. 
> 
> 1. The simplified PushPullFIFO replaces old FIFOs into one, and removes the 
> cyclic call structure between AudioDestination and AudioPullFIFO.
> 2. This refactoring will be helpful when the new WebThread for AudioWorklet
> is implemented. (push and pull operation now can be separated.)
> 
> BUG=678678
> 
> Review-Url: https://codereview.chromium.org/2549093009
> Cr-Commit-Position: refs/heads/master@{#448784}
> Committed:
https://chromium.googlesource.com/chromium/src/+/41a5385ea755ea3550ce907716c3...

TBR=rtoy@chromium.org,olka@chromium.org,haraken@chromium.org,thakis@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=678678
==========