New issue
Advanced search Search tips

Issue 896354 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Oct 22
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

IsAdScriptInStack needs to take an optional ExecutionContext* param

Project Member Reported by jkarlin@chromium.org, Oct 17

Issue description

IsAdScriptInStack asks the v8::Isolate what the current ExecutionContext is, but in some cases v8 isn't currently running (e.g., for WillSendRequest). So we need to pass the known ExecutionContext in that case.
 
Status: WontFix (was: Started)
A better way to do this is just check the ExecutionContext in WillSendRequest.

Sign in to add a comment