Bug 3774 - do renaming so that loaders are called "loader", not "client" or "delegate"
Summary: do renaming so that loaders are called "loader", not "client" or "delegate"
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 412
Hardware: Mac OS X 10.4
: P3 Enhancement
Assignee: Darin Adler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-06-30 08:14 PDT by Darin Adler
Modified: 2005-06-30 22:35 PDT (History)
0 users

See Also:


Attachments
Patch, including deleting files; best applied with cvs-apply (103.27 KB, patch)
2005-06-30 08:16 PDT, Darin Adler
sullivan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Adler 2005-06-30 08:14:08 PDT
I am planning on renaming the following classes, source files, and methods in WebKit:

    WebBaseResourceHandleDelegate -> WebLoader
    WebMainResourceClient -> WebMainResourceLoader
    WebSubresourceClient -> WebSubresourceLoader
    WebNetscapePluginConnectionDelegate -> WebNetscapePlugInStreamLoader
    subresourceClients -> subresourceLoaders
    mainClient -> mainResourceLoader
    plugInStreamClients -> plugInStreamLoaders
    _addSubresourceClient -> _addSubresourceLoader
    _removeSubresourceClient -> _removeSubresourceLoader
    _addPlugInStreamClient -> _addPlugInStreamLoader
    _removePlugInStreamClient -> _removePlugInStreamLoader
Comment 1 Darin Adler 2005-06-30 08:16:14 PDT
Created attachment 2715 [details]
Patch, including deleting files; best applied with cvs-apply

I went to the cvs server and copied the files that need renaming. This patch
now does a cvs rm on the now-obsolete files and patches all the code, including
the project, to use the new names.

Also, it fixes some patches in the WebKit project file to be group-relative and
removes one unneeded framework path. But that's all -- just the renaming and
the project file tweaks. I've compiled and tested running Safari and layout
tests.
Comment 2 Darin Adler 2005-06-30 08:17:27 PDT
Comment on attachment 2715 [details]
Patch, including deleting files; best applied with cvs-apply

This patch is simply renaming -- I considered landing it without review and
decided it would be better to get patch review.

It's a bit time-critical because we have "doubled" files in the repository due
to the "copy files on the CVS server" technique we use for renaming files in
CVS.
Comment 3 John Sullivan 2005-06-30 09:08:46 PDT
Comment on attachment 2715 [details]
Patch, including deleting files; best applied with cvs-apply

Good name change. I see that this doesn't affect any public headers, so r=me. I
am assuming that you tested Safari in some simple way.
Comment 4 Eric Seidel (no email) 2005-06-30 09:12:59 PDT
Comment on attachment 2715 [details]
Patch, including deleting files; best applied with cvs-apply

It's too bad webkit isn't using svn quite yet... or the rename of files would
have been directly supported.  Also glad to see that this is in WK and not in
WC where a rename would have further differenced us from KDE.