Bug 13212

Summary: Submit initial version of WebKit Apollo bitmap interface for feedback
Product: WebKit Reporter: Chris Brichford <chrisb>
Component: WebKit APIAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WONTFIX    
Severity: Normal CC: ap, webkit
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: Other   
OS: OS X 10.4   
Attachments:
Description Flags
Initial patch of a few web kit apollo interface files. mrowe: review-

Description Chris Brichford 2007-03-27 18:08:11 PDT
Creating a patch with this one file that defines the interfaces between WebKit Apollo and the bitmap surfaces it draws into.  The point of this patch is to get feed back on the coding style and the directory layout in WebKit/apollo.
Comment 1 Chris Brichford 2007-03-27 18:17:32 PDT
Created attachment 13838 [details]
Initial patch of a few web kit apollo interface files.

Initial patch of a few web kit apollo interface files.  Please just review.  Do not commit.
Comment 2 Chris Brichford 2007-03-27 18:24:08 PDT
There will be a source directory that contains all the implementation files for WebKitApollo at WebKit/apollo/source.  Headers private to WebKitApollo will live in WebKit/apollo/source.  Public headers will all be in WebKitApollo/apollo/include/WebKitApollo, WebKit/apollo/include/mac/WebKitApollo, WebKit/apollo/include/linux/WebKitApollo, or WebKit/apollo/include/win/WebKitApollo.

I also realized I forgot to put in a copyright notice in WebConfig.h.
Comment 3 Mark Rowe (bdash) 2007-07-15 11:23:13 PDT
Comment on attachment 13838 [details]
Initial patch of a few web kit apollo interface files.

About the directory naming:  I'm not sure what having the extra level "WebKitApollo" adds to things.  On the Mac at least the <WebKitApollo/HeaderName.h> framework-style references would work for free due to how Xcode's header copying build phase works.  You may want to take a look at how we handle the Windows WebKit layer accessing WebCore headers using the <WebCore/HeaderName.h> style import and use a similar approach here.

Miscellaneous coding style points:
*) There is a lot of extraneous whitespace inside parentheses in your typedef's.
*) The *-placement inside your typedefs isn't consistent.
*) The name m_pVTable in WebKitApolloBitmap seems like it's bordering on Hungarian notation.

What is the purpose of the WEBKIT_APOLLO_PROTO1 and WEBKIT_APOLLO_PROTO2 macros?  Their names provide no information as to the reason for their existence, and their empty definitions are not particularly enlightening either.
Comment 4 Robert Blaut 2008-07-28 10:45:13 PDT
Chris, do you plan to proceed with the bug?
Comment 5 Chris Brichford 2008-07-28 11:20:28 PDT
(In reply to comment #4)
> Chris, do you plan to proceed with the bug?
> 

Not at this time.  You can kill the bug.