[webkit-dev] Database in Worker context

Eric Uhrhane 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.
>
>     Eric
>
> 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 mailing list