[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
Thu Nov 2 02:19:54 PST 2006


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


hausmann at kde.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |enrico.qt at email.it




------- Comment #6 from hausmann at kde.org  2006-11-02 02:19 PDT -------
(In reply to comment #5)
> (From update of attachment 11298 [edit])
> 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?

Yes exactly, that is the plan for the future.

> 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.
> 

Yes, that is definitely the next step, making the ResourceHandles a bit more
intelligent themselves instead of delegating everything to the otherwise
useless 'Manager'. For the Qt/KDE port we would probably end up letting
ResourceHandleInternal inherit from QObject so we can bind it to
signal/slot/timer based network backends.

Enrico, what's the status of your patch? Shall we see about getting mine in
first as a start or are you already done anyway?


-- 
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