New issue
Advanced search Search tips

Issue 753936 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 734210



Sign in to add a comment

Remove blink::InterfaceProvider

Project Member Reported by dcheng@chromium.org, Aug 9 2017

Issue description

This is currently used to provide auto thread hopping capabilities, but each worker thread should have its own InterfaceProvideer and use that instead of Platform's.

Once that happens, we can get rid of blink::InterfaceProvider.
 
(Worker bug triage)

dcheng@: Any updates on this? It looks like each WorkerGlobalScope has its own InterfaceProvider in the current impl:
https://chromium.googlesource.com/chromium/src/+/0445e178c7e0702fcce625c09b97a37fb7a3ea32/third_party/blink/renderer/core/workers/worker_global_scope.cc#302

Comment 2 by dcheng@chromium.org, Jun 12 2018

I think there might be an issue for blink::Platform still: it looks like at least one place might still use it from a worker thread: https://cs.chromium.org/chromium/src/third_party/blink/renderer/platform/blob/blob_data.cc?rcl=676a3ff61e1a1b5c75c9dbe359b9cb38cb23b286&l=97
Cc: haraken@chromium.org
Components: Blink>Internals
This seems blink-wide architecture issue.

+Blink>Internals
cc:haraken@ for further triage

Sign in to add a comment