[webkit-reviews] review granted: [Bug 200117] Web Inspector: Storage: disable related agents when the tab is closed : [Attachment 375368] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 2 12:51:29 PDT 2019

Joseph Pecoraro <joepeck at webkit.org> has granted Devin Rousso
<drousso at apple.com>'s request for review:
Bug 200117: Web Inspector: Storage: disable related agents when the tab is

Attachment 375368: Patch


--- Comment #8 from Joseph Pecoraro <joepeck at webkit.org> ---
Comment on attachment 375368
  --> https://bugs.webkit.org/attachment.cgi?id=375368

View in context: https://bugs.webkit.org/attachment.cgi?id=375368&action=review

> Source/WebInspectorUI/UserInterface/Controllers/AppController.js:81
> +	       if (manager)
> +		   manager.activateExtraDomain(domain);

Can't wait to remove this code!

> +    networkStateUpdated(isNowOnline)

Maybe we should have a new section:

    // Protocol Callbacks

Or just:

    // ApplicationCacheObserver

And we can remove all these inner methods.

> Source/WebInspectorUI/UserInterface/Controllers/DOMStorageManager.js:105
>      itemsCleared(storageId)

Yeah, if we had:

    // DOMStorageObserver

Here that would save so much space!

> Source/WebInspectorUI/UserInterface/Views/StorageTabContentView.js:51
> -	   return !!window.DOMStorageAgent || !!window.DatabaseAgent ||
> +	   return !!(InspectorBackend.domains.ApplicationCache ||
InspectorBackend.domains.DOMStorage || InspectorBackend.domains.Database ||

We probably can't do this change until other work, since this should always be

Once we do the BackendCommands rework we can do this!

More information about the webkit-reviews mailing list