[Webkit-unassigned] [Bug 35087] New port: EFL; adding files to WebCore/*/efl

[bugzilla-daemon at webkit.org]

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


Oliver Hunt <oliver at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #48996|review?, commit-queue?      |review-
               Flag|                            |




--- Comment #12 from Oliver Hunt <oliver at apple.com>  2010-02-18 22:43:45 PST ---
(From update of attachment 48996)
The vast majority of this patch is code copied from the Gtk equivalents, really
this should be shared code.  For that reason i suggest splitting this into two
distinct patches.

One which contains the EFL changes to core WebCore types (IntRect, etc), and
one that adds the EFL specific code to the Gtk platform code.

It's possible that some of the gtk code should be renamed from (for instance)
FontPlatformDataGtk to FontPlatformDataCairo, and MediaPlayerGtk to
MediaPlayerGStreamer, or at least the common cairo and gstreamer code be split
into gstreamer and cairo specific files, with separate efl and gtk files for
those pieces of functionality that aren't sharable.  The current copy-paste
model seems likely to reduce the ability of the Efl port to benefit from work
done on the Cairo and Gtk ports.  It may be worth discussing these file
renames, etc with Gustavo, Philippe, and Ken in #webkit-gtk

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