DescriptionRevert of DevTools: Make WorkerThreadDebugger available for Worklets (patchset #8 id:360001 of https://codereview.chromium.org/2456733002/ )
Reason for revert:
The layout test animation-worklet-console.html is failing on WebKit Linux Precise Leak bot:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Precise%20Leak/builds/1566
Original issue's description:
> DevTools: Make WorkerThreadDebugger available for Worklets
>
> DevTools is now disabled on Worklets because worker's debugger infrastructure(*)
> cannot host multiple worklet execution contexts on a single thread.
> (*) Worklet has been implemented on the worker infrastructure.
>
> To support it, this CL does following things:
>
> - This expands 1:1 relationship between WorkerThreadDebugger and WorkerThread to
> 1:M relationship. The debugger has a ID-WorkerThread map to manage multiple
> WorkerThreads. WorkerThread is added when worklet context is created and
> removed when the context is destroyed.
>
> - This decouples lifetime of WorkerThreadDebugger from lifetime of worklet
> context. Before this CL, the debugger has the same lifetime of the context.
> This prevents the debugger from hosting multiple contexts. After this CL,
> the debugger has the same lifetime of WorkerBackingThread, and attaches to
> worklet context when it's created.
>
> DesignDoc: https://docs.google.com/document/d/1v0dNB6jWMe7w6OeK-Vlnn-Q4qW7KI4m97LGnMefUx1k/edit
>
> BUG=646559
>
> Committed: https://crrev.com/46d040e09230a0a04c61fc06d88636b9c3dfb627
> Cr-Commit-Position: refs/heads/master@{#430558}
TBR=dgozman@chromium.org,falken@chromium.org,haraken@chromium.org,nhiroki@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOTREECHECKS=true
NOTRY=true
BUG=646559
Committed: https://crrev.com/c023fcba8a6c9ca521c4c2a9f8921beaab18604e
Cr-Commit-Position: refs/heads/master@{#430638}
Patch Set 1 #Messages
Total messages: 9 (4 generated)
|