Commit 2dbd1cd1 authored by Matt Falkenhagen's avatar Matt Falkenhagen Committed by Commit Bot

service worker: Remove workaround for ServiceWorkerObjectHost crash.

Since issue 864731 was fixed in r576288, this workaround is no longer
needed.

Bug: 854993
Change-Id: I789dfb18cdb82ec51aa0ce5185586163c92269ec
Reviewed-on: https://chromium-review.googlesource.com/1147889Reviewed-by: default avatarKenichi Ishibashi <bashi@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#577435}
parent 961af73f
......@@ -308,16 +308,6 @@ ServiceWorkerProviderHost::~ServiceWorkerProviderHost() {
// TODO(falken): Use ScopedObserver instead of this explicit call.
RemoveAllMatchingRegistrations();
// Explicitly destroy the ServiceWorkerObjectHosts and
// ServiceWorkerRegistrationObjectHosts owned by |this|. Otherwise, this
// destructor can trigger their Mojo connection error handlers, which would
// call back into halfway destroyed |this|. This is because they are
// associated with the ServiceWorker interface, which can be destroyed while
// in this destructor (|running_hosted_version_|'s |event_dispatcher_|). See
// https://crbug.com/854993.
service_worker_object_hosts_.clear();
registration_object_hosts_.clear();
// This host may be destroyed before it received the anticipated
// HintToUpdateServiceWorker IPC from the renderer. This can occur on
// navigation failure or if the frame closed soon after navigation. The
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment