[webkit-dev] Database in Worker context
ericu at chromium.org
Thu Jan 14 14:35:34 PST 2010
Trying again; webkit-dev doesn't seem to like my @google.com address.
On Thu, Jan 14, 2010 at 2:31 PM, Eric Uhrhane <ericu at google.com> wrote:
> Dmitry requested that I not pull stuff out into DatabaseManager at
> all, so that step got skipped.
> The first changelist [currently in review] does a bunch of refactoring
> to make the database-in-workers possible, but does not yet expose any
> new functionality. I'm trying to keep the steps small, but I think it
> would have been hard to break this chunk any smaller.
> On Thu, Jan 14, 2010 at 2:24 PM, Darin Adler <darin at apple.com> wrote:
>> My main feedback on this plan would be to land and review the work in pieces that are as small as possible.
>> For example, factoring out the database bits into a separate database manager class should be done in an initial first pass that does not make other changes.
>> I’m also not sure I like classes with Manager in their name, but I suppose we have a lot of such classes (controllers, managers, etc.) in WebKit already.
>> -- Darin
More information about the webkit-dev