DescriptionRevert of Apply AllowCrossThreadAccess() in the callers of createCrossThreadTask() (patchset #9 id:160001 of https://codereview.chromium.org/1766143002/ )
Reason for revert:
Speculatively reverting this to see if it fixes a bunch of media tests on Windows that started failing as soon as this was checked in.
Started here:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7%20%28dbg%29/builds/5640
Original issue's description:
> Apply AllowCrossThreadAccess() in the callers of createCrossThreadTask()
>
> This CL removes AllowCrossThreadAccess() calls in CrossThreadTask.h and applies
> AllowCrossThreadAccess() at the callers of createCrossThreadAccess().
>
> All previous uses of createCrossThreadTask() that would require
> AllowCrossThreadAccess() are detected by compile errors
> due to lack of CrossThreadCopier for WeakPtr/pointers because
> https://codereview.chromium.org/1904283004/ removed CrossThreadCopier for
> GCed pointers.
>
> BUG=597856
>
> Committed: https://crrev.com/4c6e26cd4be4f511aa6a62e46c37178605fb3714
> Cr-Commit-Position: refs/heads/master@{#393573}
TBR=tzik@chromium.org,haraken@chromium.org,kinuko@chromium.org,yutak@chromium.org,hiroshige@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=597856
Patch Set 1 #Messages
Total messages: 9 (1 generated)
|