Revert of Avoid calling PostTask in audio callbacks

Revert of Avoid calling PostTask in audio callbacks (patchset #6 id:100001 of https://codereview.webrtc.org/2663383004/ )

Reason for revert:
Speculative revert to see if this CL caused a change in performance tests.

See https://bugs.chromium.org/p/chromium/issues/detail?id=689919 for details.

Original issue's description:
> Avoid calling PostTask in audio callbacks.
> 
> We have seen that PostTask can consume some CPU and the way we used it
> before (logging only) in the ADB is not worth the cost we see when
> profiling.
> 
> This CL simply moves frequent (trivial) stat updates from the task queue
> to the native threads to avoid calling PostTask in each callback.
> The reason for doing so before was to avoid locks but we can live without
> them since races are benign here. 
> 
> 
> BUG=webrtc:7096
> 
> Review-Url: https://codereview.webrtc.org/2663383004
> Cr-Commit-Position: refs/heads/master@{#16429}
> Committed: https://chromium.googlesource.com/external/webrtc/+/77ce9a55415a673422d424ed862be142d5e277ef

TBR=solenberg@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7096
Review-Url: https://codereview.webrtc.org/2684913003
Cr-Commit-Position: refs/heads/master@{#16490}
Committed: https://chromium.googlesource.com/external/webrtc/+/fd8f102a847e7f8bd9c719ec6cc0b5818b626705

[henrika_webrtc, 2017-02-08 12:30:30]

The CQ bit was checked by henrika@webrtc.org

[henrika_webrtc, 2017-02-08 12:30:30]

Created Revert of Avoid calling PostTask in audio callbacks

[commit-bot: I haz the power, 2017-02-08 12:30:36]

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

[the sun, 2017-02-08 13:12:21]

Weird.

[the sun, 2017-02-08 13:12:29]

lgtm

[commit-bot: I haz the power, 2017-02-08 13:23:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486557030081990, "parent_rev":
"219208991b63b868d38d4c78fc3b04fa8370b636", "commit_rev":
"fd8f102a847e7f8bd9c719ec6cc0b5818b626705"}

[commit-bot: I haz the power, 2017-02-08 13:23:18]

Description was changed from

==========
Revert of Avoid calling PostTask in audio callbacks (patchset #6 id:100001 of
https://codereview.webrtc.org/2663383004/ )

Reason for revert:
Speculative revert to see if this CL caused a change in performance tests.

See https://bugs.chromium.org/p/chromium/issues/detail?id=689919 for details.

Original issue's description:
> Avoid calling PostTask in audio callbacks.
> 
> We have seen that PostTask can consume some CPU and the way we used it
> before (logging only) in the ADB is not worth the cost we see when
> profiling.
> 
> This CL simply moves frequent (trivial) stat updates from the task queue
> to the native threads to avoid calling PostTask in each callback.
> The reason for doing so before was to avoid locks but we can live without
> them since races are benign here. 
> 
> 
> BUG=webrtc:7096
> 
> Review-Url: https://codereview.webrtc.org/2663383004
> Cr-Commit-Position: refs/heads/master@{#16429}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/77ce9a55415a673422d424ed8...

TBR=solenberg@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7096
==========

to

==========
Revert of Avoid calling PostTask in audio callbacks (patchset #6 id:100001 of
https://codereview.webrtc.org/2663383004/ )

Reason for revert:
Speculative revert to see if this CL caused a change in performance tests.

See https://bugs.chromium.org/p/chromium/issues/detail?id=689919 for details.

Original issue's description:
> Avoid calling PostTask in audio callbacks.
> 
> We have seen that PostTask can consume some CPU and the way we used it
> before (logging only) in the ADB is not worth the cost we see when
> profiling.
> 
> This CL simply moves frequent (trivial) stat updates from the task queue
> to the native threads to avoid calling PostTask in each callback.
> The reason for doing so before was to avoid locks but we can live without
> them since races are benign here. 
> 
> 
> BUG=webrtc:7096
> 
> Review-Url: https://codereview.webrtc.org/2663383004
> Cr-Commit-Position: refs/heads/master@{#16429}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/77ce9a55415a673422d424ed8...

TBR=solenberg@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7096

Review-Url: https://codereview.webrtc.org/2684913003
Cr-Commit-Position: refs/heads/master@{#16490}
Committed:
https://chromium.googlesource.com/external/webrtc/+/fd8f102a847e7f8bd9c719ec6...
==========

[commit-bot: I haz the power, 2017-02-08 13:23:19]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/fd8f102a847e7f8bd9c719ec6...

[henrika_webrtc, 2017-02-08 15:16:38]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2687573003/ by henrika@webrtc.org.

The reason for reverting is: The reason for reverting was false alarm..