Disallow UI/IO thread blocking on any other thread.

Disallow UI/IO thread blocking on any other thread.

By design, there's no ScopedAllowWait that is reachable by all code. From experience with ScopedAllowIO, it will be abused. So instead the existing callers (which should all be fixed other than two) are friends with ThreadRestrictions.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=134114

[brettw, 2012-04-25 05:41:14]

Nice LGTM

http://codereview.chromium.org/10151009/diff/8003/base/synchronization/condit...
File base/synchronization/condition_variable_win.cc (right):

http://codereview.chromium.org/10151009/diff/8003/base/synchronization/condit...
base/synchronization/condition_variable_win.cc:244: void
WinXPCondVar::TimedWait(const TimeDelta& max_time) {
Do you also want to add the assert to the XP implementation? It doesn't matter
since most developers don't use XP, but it's probably good for completeness.

[jam, 2012-04-25 22:56:46]

http://codereview.chromium.org/10151009/diff/8003/base/synchronization/condit...
File base/synchronization/condition_variable_win.cc (right):

http://codereview.chromium.org/10151009/diff/8003/base/synchronization/condit...
base/synchronization/condition_variable_win.cc:244: void
WinXPCondVar::TimedWait(const TimeDelta& max_time) {
On 2012/04/25 05:41:14, brettw wrote:
> Do you also want to add the assert to the XP implementation? It doesn't matter
> since most developers don't use XP, but it's probably good for completeness.

Done.