[Webkit-unassigned] [Bug 46021] Add NoStaticTables attributes to IDLs of FileSystem API for Worker support

[bugzilla-daemon at webkit.org]

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


David Levin <levin at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #67980|review?                     |review-
               Flag|                            |




--- Comment #2 from David Levin <levin at chromium.org>  2010-09-17 17:46:01 PST ---
(From update of attachment 67980)
View in context: https://bugs.webkit.org/attachment.cgi?id=67980&action=prettypatch

Mostly looks good. I had a few question about things that I like to understand before r+'ing. (If anyone else understands them and wants to r+, that is fine with me.)

> WebCore/ChangeLog:8
> +        Also changed their module from storage to fileapi.

Why?

> WebCore/ChangeLog:17
> +        * fileapi/ErrorCallback.idl:

A few of these idl's including this one didn't get NoStaticTables added to them. Why? (A ChangeLog comment on these file would be nice to explain it.)

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