Bug 17299

Summary: Patch providing fully-buildable Cairo-backed Win32 WebCore
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aroben, webkit
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch for Cairo support in WebCore. none

Description Brent Fulgham 2008-02-10 15:36:04 PST
The attached patch updates a few spots in WebCore that are needed to build a Cairo-based Win32 WebCore.  Additional patches are being provided for the WebKit portion of the changes.

To be completely usable, this patch requires an updated WebCore.vcproj to build the Cairo-specific parts of WebCore.  A separate bug is being created to complete this work, as some thought will need to be given to the means of supporting Cairo and CG in a single solution file.
Comment 1 Brent Fulgham 2008-02-10 15:36:44 PST
Created attachment 19051 [details]
Patch for Cairo support in WebCore.
Comment 2 Brent Fulgham 2008-02-10 15:44:21 PST
Note that this bug is related to Bug #17300 (though it does not depend on its soution).
Comment 3 Darin Adler 2008-02-10 17:59:22 PST
Comment on attachment 19051 [details]
Patch for Cairo support in WebCore.

Patch looks good. Some small issues:

 #include "config.h"
+#include "PlatformString.h"
+#include "CSSValueKeywords.h"
 #include "RenderThemeWin.h"

New includes should go in a separate group. See the <http://webkit.org/coding/coding-style.html> section called "#include statements".

+        cachedDesc->firstFamily().setFamily("Lucida Grande");

Are you sure that's the right choice? I don't think that font is included by default on Windows.
Comment 4 Robert Blaut 2008-02-11 09:20:23 PST
(In reply to comment #3)
> +        cachedDesc->firstFamily().setFamily("Lucida Grande");
> 
> Are you sure that's the right choice? I don't think that font is included by
> default on Windows.
> 

On Windows there is a similar font called Lucida Sans Unicode -> http://en.wikipedia.org/wiki/Lucida_Sans_Unicode
Comment 5 Darin Adler 2008-02-11 09:32:06 PST
Comment on attachment 19051 [details]
Patch for Cairo support in WebCore.

I'll just take this and tweak it and land it.
Comment 6 Brent Fulgham 2008-02-11 09:37:42 PST
(In reply to comment #5)
> (From update of attachment 19051 [details] [edit])
> I'll just take this and tweak it and land it.
> 

Oh, thanks!  I was going to research the proper font to use; currently it doesn't really matter because the font rendering is getting hung up somewhere else.  I'm trying to debug it today as time allows.
Comment 7 Darin Adler 2008-02-11 09:40:37 PST
Comment on attachment 19051 [details]
Patch for Cairo support in WebCore.

Committed revision 30143.