commit b50e3a5c3a69d6a6833acb7d2eecf67890d51004
author Yeunjoo Choi <ychoi@igalia.com> Wed Apr 12 01:56:59 2023
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Apr 12 01:56:59 2023
tree 80e6e31cbde20e49ed1debee45720de1769ad714
parent fdc4de4b65334c245e2cb20fa08c5d9eb7235c8b

Reland "Convert ExtensionHostMsg_RequestWorker/WorkerResponseAck to mojo"

This is a reland of commit 708ab44c7c94f08b924167ce8bcab3959c632bd0

This CL was reverted due to a failure to
ExtensionApiTestWithSwitch.ExtensionTracing. But the test has been
failed occasionally before merging the original CL even after reverting, so it seems that the original CL wasn't culprit.

Original change's description:
> Convert ExtensionHostMsg_RequestWorker/WorkerResponseAck to mojo
>
> This CL is step 2 for adding extensions.mojom.ServiceWorkerHost and
> converts ExtensionHostMsg_RequestWorker/WorkerResponseAck to mojo.
>
> ServiceWorkerHost and EventRouter for extensions should be associated
> each other and bound to the worker thread. Currently EventRouter mojo
> interface is legacy IPC associated, so we follow below steps to avoid
> flaky.
>
> Steps:
> 1) Get an associated interface for mojom::ServiceWorkerHost from the
>    IPC::SyncMessageFilter as mojom::EventRouter
> 2) Convert ExtensionHostMsg_* to mojom::ServiceWorkerHost.
> 3) Add an AssociatedInterfaceRegistry for service worker and obtain
>    intefaces for both mojom::ServiceWorkerHost and mojom::EventRouter.
>
> Bug: 1364183
> Change-Id: Id6c9d039c5dfef09fbbce628da5adb626927d4e5
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4149952
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
> Reviewed-by: Reilly Grant <reillyg@chromium.org>
> Commit-Queue: Yeunjoo Choi <ychoi@igalia.com>
> Cr-Commit-Position: refs/heads/main@{#1125212}

Bug: 1364183
Change-Id: I772fda98d7ef67b4a20e0762abc6a33c6fee1fa9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4394606
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Matthew Denton <mpdenton@chromium.org>
Commit-Queue: Yeunjoo Choi <ychoi@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1129044}