Trying again; webkit-dev doesn't seem to like my @google.com address.
On Thu, Jan 14, 2010 at 2:31 PM, Eric Uhrhane <er...@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 <da...@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 >> >> > _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
