[webkit-reviews] review requested: [Bug 30548] Remove the dependency on OriginQuotaManager from Database.cpp : [Attachment 41471] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 19 19:37:32 PDT 2009


Dumitru Daniliuc <dumi at chromium.org> has asked	for review:
Bug 30548: Remove the dependency on OriginQuotaManager from Database.cpp
https://bugs.webkit.org/show_bug.cgi?id=30548

Attachment 41471: patch
https://bugs.webkit.org/attachment.cgi?id=41471&action=review

------- Additional Comments from Dumitru Daniliuc <dumi at chromium.org>
It's OK to have DatabaseTracker.cpp depend on
OriginQuotaManager/OriginUsageRecord since we will have a Chromium-specific
implementation for DatabaseTracker.

Also, in order to keep everything thread-safe, we needed to introduce a
SecurityOrigin copy for each Database instance, that is safe to use on the DB
thread.


More information about the webkit-reviews mailing list