[Webkit-unassigned] [Bug 20393] Add WML support to WebKit

[bugzilla-daemon at webkit.org]

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


eric at webkit.org changed:

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




------- Comment #11 from eric at webkit.org  2008-08-25 20:48 PDT -------
(From update of attachment 22955)
Thank you very much for the patch George!  Unfortunately it's really
unreviewable as-is.  We can't by-pass the normal review mechanisms even for a
change off-by default.  However, we can certainly land this in pieces.  Even
landing the individual new files w/o them being compiled in or being able to
run.

There are really two questions here:
1.  Does WebKit.org want WML support?  I haven't heard any complaints against
such (and I'm myself not against adding WML support.).

2.  Is this patch OK?

The answer to the second is "it might be", but we'll be able to tell better
when split into smaller pieces which can be individually reviewed and landed
(even if not built or tested initially).

Thanks again for the patch, but I'm going to mark this large one as r-.  I look
forward to reviewing this in pieces.


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