• Dmitry Gozman's avatar
    [inspector] Prepend isolateId to remoteObjectId · 66725a53
    Dmitry Gozman authored
    This changes remoteObjectId format from
    "{injectedScriptId:123,id:456}" to "<isolateId>.<contextId>.<id>".
    
    Prepending isolateId fixes the problem that
    remote object ids clash between processes. This is especially
    troubling during cross-process navigation in Chromium, see bug.
    
    We also stop producing and parsing unnecessary json for object ids.
    
    Drive-by: fixed some tests dumping object ids. Most tests avoid
    dumping unstable values like ids, but there were few that still did.
    
    BUG=chromium:1137143
    
    Change-Id: Ia019757fb95704ccb718d3ea6cc54bde1a133382
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2461731
    Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
    Reviewed-by: 's avatarYang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#70592}
    66725a53
Name
Last commit
Last update
..
console Loading commit data...
counters Loading commit data...
cpu-profiler Loading commit data...
debugger Loading commit data...
heap-profiler Loading commit data...
runtime Loading commit data...
runtime-call-stats Loading commit data...
sessions Loading commit data...
type-profiler Loading commit data...
BUILD.gn Loading commit data...
DEPS Loading commit data...
OWNERS Loading commit data...
inspector-test.cc Loading commit data...
inspector.status Loading commit data...
isolate-data.cc Loading commit data...
isolate-data.h Loading commit data...
json-parse-expected.txt Loading commit data...
json-parse.js Loading commit data...
print-method-not-found-expected.txt Loading commit data...
print-method-not-found.js Loading commit data...
protocol-test.js Loading commit data...
task-runner.cc Loading commit data...
task-runner.h Loading commit data...
testcfg.py Loading commit data...
wasm-inspector-test.js Loading commit data...