IndexedDB No storage quota with CEF 3029 build

[kavadog]

While using indexedDB from cefsimple I keep getting warning
[2017-04-22 00:03:39.215:WARNING:quota_manager.cc(1590)] No storage quota provided in QuotaSettings.
(sample picked from https://developer.mozilla.org/en-US/doc ... _IndexedDB)
which results onerror getting called. This CL https://chromium.googlesource.com/chrom ... 3680465b3e Says we need to explicitly set QuotaSettings from "embedder" .
Same code works for chrome 56 ( 2924 build) . i.e onsuccess is called.
Tested on Windows 10 and 7. google chrome 58 works fine.

[Czarek]

Try implementing CefRequestHandler::OnQuotaRequest.

[kavadog]

OnQuotaRequest is not called for this use case. Also, it worked fine with chromium 57 and OnQuotaRequest isn't called there either. So it's definitely a regression in chromium 58 + CEF.

[Czarek]

Are you using in-memory cache?

These two issues may be related:
https://bitbucket.org/chromiumembedded/ ... mory-cache
https://bitbucket.org/chromiumembedded/ ... 3029-build

[kavadog]

I think the problem is that CEF 3029 doesn't implement the callback for the new Chromium 58 storage rules.

See 3029/chromium/src/content/public/browser/content_browser_client.cc
void ContentBrowserClient::GetQuotaSettings(
BrowserContext* context,
StoragePartition* partition,
const storage::OptionalQuotaSettingsCallback& callback) {
// By default, no quota is provided, embedders should override.
callback.Run(storage::GetNoQuotaSettings());
}

[kavadog]

I'm using cefclient test app. Also, I'm setting cache_path in my own app.

This is a change in Chromium 58 that CEF 3029 is not handling yet. CEF needs to hook up a new callback to allow the embedder (ie. CEF) to specify the QuotaSettings since as of Chromium 58, they are now defaulted to 0.

const storage::OptionalQuotaSettingsCallback& callback) {
// By default, no quota is provided, embedders should override.
callback.Run(storage::GetNoQuotaSettings());