DescriptionRevert of Avoid calling isMainThread() in the common cases when checking ScriptForbiddenScope::isScriptForbid… (patchset #2 id:20001 of https://codereview.chromium.org/2709763003/ )
Reason for revert:
meh.
Original issue's description:
> Avoid calling isMainThread() in the common cases when checking ScriptForbiddenScope::isScriptForbidden().
>
> Usually when s_scriptForbiddenCount is true we're about to crash, so check that
> first then check which thread it is. This means a background thread will read
> the potentially racy value of the counter, but it'll always return false when
> it checks isMainThread() next.
>
> Review-Url: https://codereview.chromium.org/2709763003
> Cr-Commit-Position: refs/heads/master@{#456284}
> Committed: https://chromium.googlesource.com/chromium/src/+/3b4b07ced6315a25960ca5a51dd1b7ba88de53db
TBR=dglazkov@chromium.org,jbroman@chromium.org,sigbjornf@opera.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
Review-Url: https://codereview.chromium.org/2755143002
Cr-Commit-Position: refs/heads/master@{#458085}
Committed: https://chromium.googlesource.com/chromium/src/+/36b4ac1bd5e75a1d3a37cfb7188c50cd754a3ca7
Patch Set 1 #
Messages
Total messages: 9 (5 generated)
|