Bug 144684

Summary:

Rename URL::copy() to URL::isolatedCopy() to match String.

Product:

WebKit

Reporter:

Sungmann Cho <sungmann.cho>

Component:

WebCore Misc.

Assignee:

Sungmann Cho <sungmann.cho>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none

Description Sungmann Cho 2015-05-06 06:25:33 PDT

SSIA

Comment 1 Sungmann Cho 2015-05-06 07:03:15 PDT

Created attachment 252471 [details]
Patch

Comment 2 Darin Adler 2015-05-06 09:26:10 PDT

Comment on attachment 252471 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=252471&action=review

> Source/WebCore/platform/URL.h:86
>      // no other reason to ever prefer copy() over plain old assignment.

Should fix this comment too.

Comment 3 Sungmann Cho 2015-05-06 15:05:14 PDT

Created attachment 252523 [details]
Patch

Comment 4 Sungmann Cho 2015-05-06 15:05:43 PDT

Addressed darin's comments.

Comment 5 WebKit Commit Bot 2015-05-06 17:29:44 PDT

Comment on attachment 252523 [details]
Patch

Clearing flags on attachment: 252523

Committed r183901: <http://trac.webkit.org/changeset/183901>

Comment 6 WebKit Commit Bot 2015-05-06 17:29:47 PDT

All reviewed patches have been landed.  Closing bug.