Change FrameBuffer::Stop to not require a critical section.

Change FrameBuffer::Stop to not require a critical section.

BUG=webrtc:7331
Review-Url: https://codereview.webrtc.org/2749563002
Cr-Commit-Position: refs/heads/master@{#17228}
Committed: https://chromium.googlesource.com/external/webrtc/+/0a735643388622494cd59720a40b51899985be99

[tommi, 2017-03-13 14:47:34]

Remove extra check

[tommi, 2017-03-13 14:48:07]

tommi@webrtc.org changed reviewers:
+ philipel@webrtc.org

[tommi, 2017-03-13 14:48:08]

ptal

[tommi, 2017-03-13 14:48:13]

The CQ bit was checked by tommi@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-13 14:48:20]

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

[tommi, 2017-03-13 14:56:14]

Restore the Reset() call

[tommi, 2017-03-13 20:31:43]

The CQ bit was checked by tommi@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-13 20:31:52]

Dry run: 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-03-13 20:53:34]

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

[commit-bot: I haz the power, 2017-03-13 20:53:35]

Dry run: This issue passed the CQ dry run.

[philipel, 2017-03-14 09:51:48]

This might solve the crash, but I would like to know what is causing the
underlying problem.

The lock should be free the vast majority of the time, I will look into it.

[tommi, 2017-03-14 12:28:39]

Add comment and link to bug

[tommi, 2017-03-14 12:29:17]

On 2017/03/14 09:51:48, philipel wrote:
> This might solve the crash, but I would like to know what is causing the
> underlying problem.
> 
> The lock should be free the vast majority of the time, I will look into it.

Comment and link to bug added as discussed offline.

[tommi, 2017-03-14 12:29:25]

The CQ bit was checked by tommi@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-14 12:29:35]

Dry run: 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-03-14 13:02:13]

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

[commit-bot: I haz the power, 2017-03-14 13:02:14]

Dry run: This issue passed the CQ dry run.

[philipel, 2017-03-14 13:17:51]

lgtm

[tommi, 2017-03-14 13:21:40]

The CQ bit was checked by tommi@webrtc.org

[commit-bot: I haz the power, 2017-03-14 13:21:43]

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-03-14 13:23:57]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1489497700280510,
"parent_rev": "2cb58fd162e383a11180cc3fffc20fc1018ae525", "commit_rev":
"0a735643388622494cd59720a40b51899985be99"}

[commit-bot: I haz the power, 2017-03-14 13:24:00]

Description was changed from

==========
Change FrameBuffer::Stop to not require a critical section.

BUG=webrtc:7331
==========

to

==========
Change FrameBuffer::Stop to not require a critical section.

BUG=webrtc:7331

Review-Url: https://codereview.webrtc.org/2749563002
Cr-Commit-Position: refs/heads/master@{#17228}
Committed:
https://chromium.googlesource.com/external/webrtc/+/0a735643388622494cd59720a...
==========

[commit-bot: I haz the power, 2017-03-14 13:24:01]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/external/webrtc/+/0a735643388622494cd59720a...