[Webkit-unassigned] [Bug 26103] [GTK] Remove gtk/cairo/pango/freetype/fontconfig includes from the header files in WebCore

[bugzilla-daemon at webkit.org]

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


zecke at selfish.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[GTK] Remove                |[GTK] Remove
                   |gtk/cairo/pango/freetype/fon|gtk/cairo/pango/freetype/fon
                   |tconfig includes from the   |tconfig includes from the
                   |header files                |header files in WebCore




------- Comment #3 from zecke at selfish.org  2009-05-31 19:37 PDT -------
(In reply to comment #2)
> I wonder whether this approach can result in any noticable compile time
> improvement at all (e.g. many source files in applications that include webkit
> headers anyway already include gtk.h).
> 
> Do you have any proof that this actually speeds up the build time of
> applications?

I don't have any proof but the goal is not to speed up build time of
applications using WebKit/GTK+. The goal is to speed up compiling of WebKit
itself. I have changed the summary to explicitly mention WebCore. As of today
WebCore/css/MediaQueryEvaluator.cpp is including libsoup/soup.h. This will
result in including glib-object, glib. I had a hard time finding any file in
WebCore that doesn't include any of gtk/cairo/pango/freetype/fontconfig.

Let me try to produce some numbers...


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