Revert "v8binding: Do not hold a cross origin ScriptState in IDL callback function"

This reverts commit 756bea38c853ce40e3daba7f7fadf85b0920785f.

Reason for revert: FindIt suspects that this is the culprit for a number of failures

Original change's description:
> v8binding: Do not hold a cross origin ScriptState in IDL callback function
> Make IDL callback function not hold a ScriptState of its
> creation context when it's cross origin from the incumbent
> realm.
> Not holding a cross origin ScriptState, there is much
> less risk to access a cross origin context.
> IDL callback interface will follow the same approach in
> a separate patch.
> Bug: 892755, 886588, 904218
> Change-Id: Ie55b436fcc5f66f4ee053ef08ad98ea68fb3a2d6
> Reviewed-on:
> Reviewed-by: Kentaro Hara <>
> Reviewed-by: Hitoshi Yoshida <>
> Commit-Queue: Yuki Shiino <>
> Cr-Commit-Position: refs/heads/master@{#609662},,

Change-Id: Ic0e5a3006a43f8a95202ac1d890f365307068877
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 892755, 886588, 904218
Reviewed-by: Maxim Kolosovskiy <>
Commit-Queue: Maxim Kolosovskiy <>
Cr-Commit-Position: refs/heads/master@{#609678}
28 files changed