[Webkit-unassigned] [Bug 18435] Cairo build does not work as it is in the tree

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Apr 14 09:04:27 PDT 2008


aroben at apple.com changed:

           What    |Removed                     |Added
  Attachment #20487|review?                     |review-
               Flag|                            |

------- Comment #2 from aroben at apple.com  2008-04-14 09:04 PDT -------
(From update of attachment 20487)
This patch seems to use tabs for indentation. Please only use 4 spaces for

@@ -1665,6 +1666,7 @@ HRESULT STDMETHODCALLTYPE WebFrame::spoo
     /* [in] */ UINT endPage,
     /* [retval][out] */ void* ctx)
     if (!printDC || !ctx) {
         return E_POINTER;
@@ -1748,8 +1750,11 @@ HRESULT STDMETHODCALLTYPE WebFrame::spoo
     return S_OK;
+       notImplemented();
+       return E_NOTIMPL;
+#endif //!PLATFORM(CAIRO)

Can we move most of spoolPages down into WebCore? You should talk to Alp Toker,
who I think was working on a cross-platform printing abstraction.

 void DrawTextAtPoint(CGContextRef cgContext, LPCTSTR text, int length, POINT
point, const WebFontDescription& description, CGColorRef color, int
underlinedIndex, bool drawAsPassword)
     GraphicsContext context(cgContext);

     String drawString(text, length);
     if (drawAsPassword)
         drawString = drawString.impl()->secure(WebCore::bullet);
     WebCoreDrawTextAtPoint(context, drawString, point, makeFont(description),
color, underlinedIndex);
+       notImplemented();
+#endif //!Platform(CAIRO)

Seems like this whole function will have to be compiled out for Cairo, since
you won't have a CGContextRef type.

I think we can/should rework WebDragClient to use the new ImageBuffer
abstraction so that we can remove the CG-specific code from it.

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