Bug 118584 – [Application Cache] Assert failed if ApplicationCacheStorage::setCacheDirectory is called more than once.

• Component: WebCore Misc.
• Status: RESOLVED INVALID

[Donggwan Kim]

ApplicationCacheStorage::setCacheDirectory could be called more than once.
So below assert sentence should be removed.
ASSERT(m_cacheDirectory.isNull());

[Donggwan Kim]

Created attachment 206508 [details]
Patch

[Alexey Proskuryakov]

Can you please explain why it can be called multiple times? The assertion checks for exactly the opposite, that this shouldn't happen.

[Donggwan Kim]

I think following cases could be happened

1. when storeCopyOfCache is called for some reason after setCacheDirectory was called to initialize default cache directory by application
2. For test, it could be called more than once. For example, it was called two times in test_ewk_setting.cpp of Efl port.

In my humble opinion, there is no reason for limiting setCacheDirectory force to be called only one time. In other cases, (e.g. SQLDatabase, Local Storage ...) there are no limitation like this.

So i think it should be removed.

If you have any other opinion, please let me know.

[Alexey Proskuryakov]

Comment on attachment 206508 [details]
Patch

If we are to allow changing directory on the fly, then we should ensure that on-disk database state doesn't get out of sync. So, we should either close it and open one from the new directory, or make this function assert when called in a situation where safe change is not possible.

[Donggwan Kim]

Ok. I understood what you mean. I will close this patch as Won't fix.

[Donggwan Kim]

I think INVALID is more reasonable to this patch