Revert "Reland "[DevTools] Application>Clear Site Data: stop service worker when unregistering it""

This reverts commit 46584179.

Reason for revert: Still breaks http/tests/devtools/application-panel/storage-view-reports-quota-and-clear.js on Mac, see
https://ci.chromium.org/p/chromium/builders/ci/Mac10.12%20Tests

Original change's description:
> Reland "[DevTools] Application>Clear Site Data: stop service worker when unregistering it"
> 
> This is a reland of 3dc09a0d
> 
> Original change's description:
> > [DevTools] Application>Clear Site Data: stop service worker when unregistering it
> > 
> > We should stop the service worker before unregistering it.
> > 
> > Originally, clicking "Clear site data" will unregister the service worker. In
> > Application>Service Workers, the service worker's tab title is marked as
> > "deleted", but its "status" says it's still running.
> > 
> > Bug: 894783
> > Change-Id: Ib1550cfab8274a27495bb795501f54e9a19f6a36
> > Reviewed-on: https://chromium-review.googlesource.com/c/1480725
> > Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
> > Commit-Queue: Haihong Li (Harley) <hhli@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#635263}
> 
> Bug: 894783
> Change-Id: I4b1b9a2f3764391846e47f8752cab6e223adbb1d
> Reviewed-on: https://chromium-review.googlesource.com/c/1490432
> Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
> Commit-Queue: Haihong Li (Harley) <hhli@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#636612}

TBR=dgozman@chromium.org,pfeldman@chromium.org,hhli@chromium.org

Change-Id: I3935ddfc1374e418e4ec935e3387dabf8c183161
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 894783
Reviewed-on: https://chromium-review.googlesource.com/c/1495979Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636747}

third_party/blink/renderer/devtools/front_end/resources/ClearStorageView.js

@@ -183,20 +183,6 @@ Resources.ClearStorageView = class extends UI.ThrottledWidget {
         appcacheModel.reset();
     }
 
-    if (set.has(Protocol.Storage.StorageType.Service_workers) || hasAll) {
-      for (const serviceWorkerManager of SDK.targetManager.models(SDK.ServiceWorkerManager)) {
-        const securityOriginManager = serviceWorkerManager.target().model(SDK.SecurityOriginManager);
-        for (const registration of serviceWorkerManager.registrations().values()) {
-          if (!securityOriginManager.securityOrigins().includes(registration.securityOrigin))
-            continue;
-          const activeVersion = registration.versionsByMode().get(SDK.ServiceWorkerVersion.Modes.Active);
-          if (!activeVersion)
-            continue;
-          serviceWorkerManager.stopWorker(activeVersion.id);
-        }
-      }
-    }
-
     this._clearButton.disabled = true;
     const label = this._clearButton.textContent;
     this._clearButton.textContent = Common.UIString('Clearing...');

third_party/blink/web_tests/http/tests/devtools/application-panel/storage-view-reports-quota-and-clear-expected.txt → third_party/blink/web_tests/http/tests/devtools/application-panel/storage-view-reports-quota-expected.txt

@@ -2,8 +2,6 @@ Tests quota reporting.
 
 Tree element found: true
 Clear storage view is visible: true
-
-Running: Clear all data
 -- B used out of -- storage quota
 Usage breakdown:
 
@@ -11,10 +9,4 @@ Running: Now with data
 -- KB used out of -- storage quota
 Usage breakdown:
 IndexedDB: --.- KB
-Service Workers: --.- B
-
-Running: Clear all data, again
-service worker is stopping
--- B used out of -- storage quota
-Usage breakdown:
 

third_party/blink/web_tests/http/tests/devtools/application-panel/storage-view-reports-quota-and-clear.js → third_party/blink/web_tests/http/tests/devtools/application-panel/storage-view-reports-quota.js

@@ -13,7 +13,7 @@
 
   var updateListener = null;
 
-  async function writeToIndexdDB() {
+  async function writeArray() {
     var array = [];
     for (var i = 0; i < 20000; i++)
       array.push(i % 10);
@@ -51,23 +51,13 @@
           typeUsage = children[j].textContent + typeUsage;
         if (children[j].classList.contains('usage-breakdown-legend-value')) {
           // Clean usage value because it's platform-dependent.
-          var cleanedValue = children[j].textContent.replace(/\d+(.\d+)?\sKB/, '--.- KB')
-                                                    .replace(/\d+(.\d+)?\sB/, '--.- B');
+          var cleanedValue = children[j].textContent.replace(/\d+.\d\sKB/, '--.- KB');
           typeUsage = typeUsage + cleanedValue;
         }
       }
       TestRunner.addResult(typeUsage);
     }
   }
-
-  function isServiceWorkerStopping(registration) {
-    const version = registration.versionsByMode().get(SDK.ServiceWorkerVersion.Modes.Redundant);
-    if (!version)
-      return null;
-    const status = version ? version.runningStatus : null;
-    return status === 'stopping';
-  }
-
   UI.viewManager.showView('resources');
 
   var parent = UI.panels.resources._sidebar._applicationTreeElement;
@@ -79,33 +69,13 @@
   var clearStorageView = UI.panels.resources.visibleView;
   TestRunner.addResult('Clear storage view is visible: ' + (clearStorageView instanceof Resources.ClearStorageView));
 
-  TestRunner.markStep('Clear all data');
-
   clearStorageView._clearButton.click();
   await dumpWhenMatches(clearStorageView, usage => usage === 0);
 
   TestRunner.markStep('Now with data');
 
-  await writeToIndexdDB();
-
-  var scriptURL = 'http://127.0.0.1:8000/devtools/service-workers/resources/service-worker-empty.js';
-  var scope = 'http://127.0.0.1:8000/devtools/service-workers/resources/scope/';
-  await ApplicationTestRunner.registerServiceWorker(scriptURL, scope);
+  await writeArray();
   await dumpWhenMatches(clearStorageView, usage => usage > 20000);
 
-  TestRunner.markStep('Clear all data, again');
-
-  for (const serviceWorkerManager of SDK.targetManager.models(SDK.ServiceWorkerManager)) {
-    serviceWorkerManager.addEventListener(
-    SDK.ServiceWorkerManager.Events.RegistrationUpdated, (event) => {
-      if (isServiceWorkerStopping(event.data))
-        TestRunner.addResult('service worker is stopping');
-      }, this);
-  }
-
-  clearStorageView._clearButton.click();
-
-  await dumpWhenMatches(clearStorageView, usage => usage === 0);
-
   TestRunner.completeTest();
 })();