[Webkit-unassigned] [Bug 45808] Add Worker support for FileSystem API

Mon Sep 20 12:49:15 PDT 2010

https://bugs.webkit.org/show_bug.cgi?id=45808

--- Comment #18 from David Levin <levin at chromium.org>  2010-09-20 12:49:14 PST ---
(In reply to comment #15)
> > > WebCore/ChangeLog:1454
> > > +        Added NoStaticTables and changed modulename from storage to fileapi:
> > 
> > This change seems very misplaced.
> 
> Hmm that's true I removed them from this patch.  I'm going to file another issue.

Actually I didn't mean that you needed a new patch. I mean the description in the ChangeLog was very misplaced. (It was ~100 lines below your change log entry in some random spot.) But I'm glad that you created a new patch which was really small.

As much as you can break down your big patches into smaller ones, they will be easier to review, more thoroughly reviewed and each one will get through more quickly (though on the whole it may take a little longer).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.