Commit 26b447dd authored by Patrick Monette's avatar Patrick Monette Committed by Commit Bot

Stop initializing SharedWorkerWatcher instances

There's an issue with how SharedWorkerWatcher handles shared workers
with a file: URL. This CL avoids reverting the bigger CL that introduce
the SharedWorkerWatcher class while still avoiding the issue.

Bug: 1013168
Change-Id: I4524c4c8b1ae85b56ec22bc49e0f7ddb0bd5a27b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852944
Commit-Queue: Patrick Monette <pmonette@chromium.org>
Reviewed-by: default avatarSigurður Ásgeirsson <siggi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#704647}
parent 176de16d
...@@ -127,13 +127,11 @@ void ChromeBrowserMainExtraPartsPerformanceManager::Observe( ...@@ -127,13 +127,11 @@ void ChromeBrowserMainExtraPartsPerformanceManager::Observe(
const content::NotificationDetails& details) { const content::NotificationDetails& details) {
switch (type) { switch (type) {
case chrome::NOTIFICATION_PROFILE_CREATED: { case chrome::NOTIFICATION_PROFILE_CREATED: {
Profile* profile = content::Source<Profile>(source).ptr(); // TODO(1013168): Create the worker watchers here.
CreateSharedWorkerWatcher(profile);
break; break;
} }
case chrome::NOTIFICATION_PROFILE_DESTROYED: { case chrome::NOTIFICATION_PROFILE_DESTROYED: {
Profile* profile = content::Source<Profile>(source).ptr(); // TODO(1013168): Delete the worker watchers here.
DeleteSharedWorkerWatcher(profile);
break; break;
} }
default: default:
......
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