provide .clearCache() functionality

[joeyvandijk]

To prevent data is cached from a previous browser session provide the functionality to clear the cache whenever needed.

[joelgriffith]

Probably should think about other types of caches as well: cookies, local-storage, indexDB and more.

[joeyvandijk]

@joelgriffith good point, but ApplicationCache & CacheStorage are now experimental and I can't find if they are a more specific way of managing cache or only meant for local-storage, etc?

Talking about IndexedDB & Serviceworker, those features do not call their data cache in any way and therefore I think we should name them as they are mentioned in the Devtools protocol documentation.
Perhaps we should make this clear in the documentation around cacheClear what it does(n't) do! But this is what I know, can find, so perhaps people can later on (here or with other PRs) add this information?

[joelgriffith]

Sorry to keep this open for so long, but I think we're wanting to name things with verb first, so in this case clearCache. Could you make this change? Otherwise looks great!

[adieuadieu]

Thank you, @joeyvandijk!

[joeyvandijk]

👍 Same applies for PR 123 btw 😉 or do you want the name change also there?

[adieuadieu]

@joeyvandijk Yes please! We're settling on a verbNoun() API method naming convention.

[joeyvandijk]

@joelgriffith can you review again? ;)

[adieuadieu]

@joeyvandijk Sorry, another thing:

Talking about IndexedDB & Serviceworker, those features do not call their data cache in any way and therefore I think we should name them as they are mentioned in the Devtools protocol documentation.
Perhaps we should make this clear in the documentation around cacheClear what it does(n't) do! But this is what I know, can find, so perhaps people can later on (here or with other PRs) add this information?

As part of this PR, could you add a note about that in the API documentation? Just to set peoples expectations about which cache gets cleared (and which things don't get cleared.)

[joeyvandijk]

@joelgriffith @adieuadieu have updated, but am not 100% sure if I am right, while the documentation is quite limited in its explanations. But thats why PRs exists 😉 to update it with the correct information if I am wrong.

[adieuadieu]

Thank you, @joeyvandijk! This looks good.

[joeyvandijk]

You're welcome 👍 😄