Bug 22805 - Implement image/URL dragging for Windows Cairo backend
Summary: Implement image/URL dragging for Windows Cairo backend
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-10 23:14 PST by Brent Fulgham
Modified: 2008-12-12 22:32 PST (History)
0 users

See Also:


Attachments
Patch implementing Windows Cairo image drag. (8.70 KB, patch)
2008-12-10 23:33 PST, Brent Fulgham
no flags Details | Formatted Diff | Diff
Use GraphicContext types, rather than CG-specific calls. (9.46 KB, patch)
2008-12-11 22:15 PST, Brent Fulgham
oliver: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2008-12-10 23:14:21 PST
The DragImageCairoWin.cpp was basically a stub that didn't do anything.  The attached patch provides an implementation of the corresponding CG code in terms of the Cairo API.  This allows URL drag images to appear, as well as enabling the dragging of images.
Comment 1 Brent Fulgham 2008-12-10 23:33:58 PST
Created attachment 25942 [details]
Patch implementing Windows Cairo image drag.
Comment 2 Oliver Hunt 2008-12-10 23:41:43 PST
Comment on attachment 25942 [details]
Patch implementing Windows Cairo image drag.

Do we have a typedef that encapsulates CGContextRef and struct _cairo* ?

If we do we can remove the icky ifdefs in WebDragClient.cpp, but i think it would be much better if we could get this code to  work in terms of GraphicsContext
Comment 3 Brent Fulgham 2008-12-11 10:29:06 PST
(In reply to comment #2)
> (From update of attachment 25942 [details] [review])
> Do we have a typedef that encapsulates CGContextRef and struct _cairo* ?
> 
> If we do we can remove the icky ifdefs in WebDragClient.cpp, but i think it
> would be much better if we could get this code to  work in terms of
> GraphicsContext

There are large regions of WebKit/win that are coded in terms of CoreGraphics primitives, that are just as applicable to the Cairo back-end.

I would *love* to revise these to use GraphicsContext instead -- but will that be acceptable to the Apple side of things?

This is maybe a question for Darin... 

Comment 4 Brent Fulgham 2008-12-11 22:15:16 PST
Created attachment 25974 [details]
Use GraphicContext types, rather than CG-specific calls.
Comment 5 Oliver Hunt 2008-12-11 22:22:52 PST
Comment on attachment 25974 [details]
Use GraphicContext types, rather than CG-specific calls.

r=me
Comment 6 Matt Lilek 2008-12-12 22:32:46 PST
Committed revision 39270.