[webkit-reviews] review granted: [Bug 22465] Define KURL::copy() and use it in WorkerThread.cpp : [Attachment 25456] v1 patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Nov 24 22:14:31 PST 2008


Geoffrey Garen <ggaren at apple.com> has granted Darin Fisher (Google)
<darin at chromium.org>'s request for review:
Bug 22465: Define KURL::copy() and use it in WorkerThread.cpp
https://bugs.webkit.org/show_bug.cgi?id=22465

Attachment 25456: v1 patch
https://bugs.webkit.org/attachment.cgi?id=25456&action=review

------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
"deepCopy" might be a better name for conveying that you shouldn't call this
function if you don't mean it. Then you can remove this part of you comment,
too:

// Makes a deep copy.

r=me


More information about the webkit-reviews mailing list