[webkit-reviews] review requested: [Bug 38503] Remove all custom DB callbacks in favor of auto-generated callbacks : [Attachment 55068] patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue May 4 16:07:35 PDT 2010
Dumitru Daniliuc <dumi at chromium.org> has asked for review:
Bug 38503: Remove all custom DB callbacks in favor of auto-generated callbacks
https://bugs.webkit.org/show_bug.cgi?id=38503
Attachment 55068: patch
https://bugs.webkit.org/attachment.cgi?id=55068&action=review
------- Additional Comments from Dumitru Daniliuc <dumi at chromium.org>
Should fix the GTK bot problem.
(In reply to comment #5)
> (From update of attachment 55006 [details])
> + * Copyright (C) 2007 Apple Inc. All rights reserved.
>
> Apple?
Oops, fixed. I assume you're referring to the new IDL files?
> + $implIncludes{"JS${type}.h"} = 1;
>
> This is ugly.
Not sure how to handle this better. Most generated code needs to include
${type}.h, and callbacks need to include JS${type}.h for the toJS() function.
More information about the webkit-reviews
mailing list