commit e88c95e516b6882be1c61b3a9eb334e55ff70e07
author Henrik Boström <hbos@webrtc.org> Wed Jul 08 11:18:50 2020 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Jul 08 10:55:30 2020 +0000
tree 55f803ef8037662fdd5640cd46e89675411ecfc5
parent b504780c42a006bc368b94755427cece59f2f42e

[Stats] Add more rtc::Thread::ScopedDisallowBlockingCalls to getStats().

This ensures with DCHECK-crashes that we don't accidentally do more
blocking invokes than we think.

Remaining blocking invokes FYI:
- PrepareTransceiverStatsInfos_s_w() does 1 blocking invoke (regardless
  of the number of transceivers or channels) to the worker thread. This
  is because VoiceMediaChannel, VideoMediaChannel and GetParameters()
  execute on the worker thread, and the result of these operations are
  needed on the signalling thread.
- pc_->GetCallStats() does 1 blocking invoke to the worker thread.

These two blocking invokes can be merged, reducing the total number of
blocking invokes from 2 to 1, but this CL does not attempt to do that.
I filed https://crbug.com/webrtc/11767 for that.

Bug: webrtc:11716
Change-Id: Iebc2ab350d253fd037211cdd283825b4e5b2d446
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178867
Reviewed-by: Evan Shrubsole <eshr@google.com>
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31670}