Reland of Avoid calling PostTask in audio callbacks

Reland of Avoid calling PostTask in audio callbacks (patchset #1 id:1 of https://codereview.webrtc.org/2684913003/ )

Reason for revert:
The reason for reverting was false alarm.

Original issue's description:
> 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

TBR=solenberg@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7096
Review-Url: https://codereview.webrtc.org/2687573003
Cr-Commit-Position: refs/heads/master@{#16497}
Committed: https://chromium.googlesource.com/external/webrtc/+/87d11cdbca721c93cf3856026e1bce02c21abdff

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

The CQ bit was checked by henrika@webrtc.org

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

Created Reland of Avoid calling PostTask in audio callbacks

[commit-bot: I haz the power, 2017-02-08 15:16:50]

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

[commit-bot: I haz the power, 2017-02-08 15:16:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486566999065320, "parent_rev":
"5d83780c42100eaddafc4b83fa453a4c9d936e7a", "commit_rev":
"87d11cdbca721c93cf3856026e1bce02c21abdff"}

[commit-bot: I haz the power, 2017-02-08 15:16:59]

Description was changed from

==========
Reland of Avoid calling PostTask in audio callbacks (patchset #1 id:1 of
https://codereview.webrtc.org/2684913003/ )

Reason for revert:
The reason for reverting was false alarm.

Original issue's description:
> 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...

TBR=solenberg@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7096
==========

to

==========
Reland of Avoid calling PostTask in audio callbacks (patchset #1 id:1 of
https://codereview.webrtc.org/2684913003/ )

Reason for revert:
The reason for reverting was false alarm.

Original issue's description:
> 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...

TBR=solenberg@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7096

Review-Url: https://codereview.webrtc.org/2687573003
Cr-Commit-Position: refs/heads/master@{#16497}
Committed:
https://chromium.googlesource.com/external/webrtc/+/87d11cdbca721c93cf3856026...
==========

[commit-bot: I haz the power, 2017-02-08 15:17:00]

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

[the sun, 2017-02-08 15:57:48]

lgtm