[Webkit-unassigned] [Bug 27425] adding auto-generator support for GDOMHTMLElementWrapperFactory[.cpp/.h]

[bugzilla-daemon at webkit.org]

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


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34060|review?(eric at webkit.org)    |review-
               Flag|                            |




--- Comment #21 from Eric Seidel <eric at webkit.org>  2009-08-04 08:51:56 PDT ---
(From update of attachment 34060)
I wasn't clear, sorry.  I agree that JS should be default.  I just wanted all
the places where you assumed !JS was Gdom to turn into explicit GDom checks.

passing no --bindingType can/should default to JS.  Otherwise you'd have to
change lots more files here. :)

327         if ($binding eq 'Gdom') {
 328             # skip DATAGRID for now
 329             next if ($conditional eq 'DATAGRID');
 330         }

Those can all go away if you just remove DATAGRID from the list of tags at the
start, no?  Removing it once instead of skipping it in every for loop seems
like a cleaner solution, or?  Also, don't you all have ENABLE_DATAGRID off
anyway?  Won't make_names just print #defines around all the DATAGRID stuff
which will be compiled out?

$F really should have a better name, per our style guidelines:
http://webkit.org/coding/coding-style.html.  Since you're just moving existing
code, I guess it's OK as is though.

So the GNUMakefile.am changes are fine, but not required if you restore the JS
default.

This in general looks great.  I think we still can fix the DATAGRID with a
cleaner hack.

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