[Webkit-unassigned] [Bug 11467] patch to make the kde support optional in the Qt build

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Nov 1 15:17:27 PST 2006


http://bugs.webkit.org/show_bug.cgi?id=11467


mjs at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #11298|review?                     |review+
               Flag|                            |




------- Comment #5 from mjs at apple.com  2006-11-01 15:17 PDT -------
(From update of attachment 11298)
r=me

Have you considered making a platform/kde directory and factoring some of this
so that there can be kde and pure-qt versions of some things?

Also, I would suggest refactoring the qt port to not need
ResourceHandleManager, the Qt and KDE versions of ResourceHandle could directly
each handle an underlying platform loader. This is the direction things are
moving in for the other ports. This is ok to land though.


-- 
Configure bugmail: http://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.



More information about the webkit-unassigned mailing list