FR: Allow auto-cleaning on browser termination

[Gitoffthelawn]

Hello Andy. :)

Can you add an option to perform the cleaning when the browser terminates (well, obviously, just before it terminates)?

[andy-portmen]

As far as I know, there is no event to listen for that reliably calls your observer when the browser is about to exit. Note that to have no effect on the browser performance, this extension uses "event page" background script; https://github.com/andy-portmen/ecleaner/blob/master/manifest.json#L21 (this is not yet supported on Firefox!)

This means there is no persistent background script at all. We need to rely on the Chrome's runtime events and there is no exit event as far as I can tell

[andy-portmen]

So in the latest dev version, the extension performs the last cleaning when there is no browser window open. This is the closest to what you are looking for. Note that the option is disabled by default and can be turned on from the options page.

Also, note that you need to perform at least one manual cleaning from the popup so that the extension could record whatever you want to be cleaned.

[Gitoffthelawn]

Thanks Andy!

Ah, the (current) limitations of WebExtensions API.

Just FYI, this extension can delete everything on Firefox termination: https://addons.mozilla.org/firefox/addon/clickclean/

Of course, it does not use WebExtensions API.

Also, note that you need to perform at least on manual cleaning from the popup so that the extension could record what even you want to be cleaned.

Hmmm.... can an options page allow this be set ahead of time?

[andy-portmen]

I meant "one manual cleaning". This is to know what is the preferred cleaning preferences of the user.

[Gitoffthelawn]

I figured that's what you meant. I wonder if you can create an options menu to specify what should be cleaned on termination so that one manual cleaning need not be performed.

[andy-portmen]

Yes, you can now set whatever you prefer to be cleaned on exit from the panel without actually performing the cleaning

[Gitoffthelawn]

Thanks!

BTW, does it really clean IndexedDB Databases? I thought that wasn't possible (yet) with WebExtensions.

[andy-portmen]

IndexedDB is removed from the latest release (only for FF)

[Gitoffthelawn]

I think it is on the Mozilla schedule to make IndexedDB clearable... perhaps for version 58.

[andy-portmen]

since FF throws error, I had to remove these types.
https://github.com/andy-portmen/ecleaner/blob/master/data/panel/index.js#L9-L11

Let me know when it is supported and I'll remove it from the list.

[Gitoffthelawn]

I created a new issue for this new topic: #10