[webkit-reviews] review granted: [Bug 31440] Add Chromium's DatabaseTracker implementation to WebKit : [Attachment 43133] patch

[bugzilla-daemon at webkit.org]

Dimitri Glazkov (Google) <dglazkov at chromium.org> has granted Dumitru Daniliuc
<dumi at chromium.org>'s request for review:
Bug 31440: Add Chromium's DatabaseTracker implementation to WebKit
https://bugs.webkit.org/show_bug.cgi?id=31440

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

------- Additional Comments from Dimitri Glazkov (Google)
<dglazkov at chromium.org>
I don't like how DatabaseTracker is so fragmented by the ifdefs. It's fine for
now, but ideally this really needs to be abstracted by purpose and the lines of
implementation drawn along that. Can you file a bug on this?