Bug 78108

Summary:

Rename [ConvertNullStringTo=] to [TreatReturnedNullStringAs=]

Product:

WebKit

Reporter:

Kentaro Hara <haraken>

Component:

WebCore JavaScript

Assignee:

Kentaro Hara <haraken>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, darin, japhet, macpherson, menard, ojan, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

77393

Attachments:

Description	Flags
Patch	abarth: review+, webkit.review.bot: commit-queue-
rebased patch for commit	haraken: commit-queue-
rebased patch for commit	none

Description Kentaro Hara 2012-02-08 05:47:22 PST

[ConvertNullStringTo=] is not descriptive. To clarify that it specifies the behavior when the null string is returned by WebCore, we can rename [ConvertNullStringTo=] to [TreatReturnedNullStringAs=], also for naming consistency with [TreatNullAs] and [TreatUndefinedAs].

Comment 1 Kentaro Hara 2012-02-08 05:50:13 PST

Created attachment 126067 [details]
Patch

Comment 2 WebKit Review Bot 2012-02-08 10:37:15 PST

Comment on attachment 126067 [details]
Patch

Rejecting attachment 126067 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
torage/IDBRequest.idl
patching file Source/WebCore/storage/Storage.idl
patching file Source/WebCore/storage/StorageEvent.idl
patching file Source/WebCore/websockets/WebSocket.idl
patching file Source/WebCore/xml/XMLHttpRequest.idl
patching file Source/WebCore/xml/XPathNSResolver.idl
patching file Source/WebCore/xml/XSLTProcessor.idl

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Adam Barth']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue/

Full output: http://queues.webkit.org/results/11449493

Comment 3 Kentaro Hara 2012-02-08 16:04:02 PST

Created attachment 126173 [details]
rebased patch for commit

Comment 4 WebKit Review Bot 2012-02-08 22:34:31 PST

Comment on attachment 126173 [details]
rebased patch for commit

Rejecting attachment 126173 [details] from commit-queue.

New failing tests:
platform/chromium/compositing/layout-width-change.html
Full output: http://queues.webkit.org/results/11487095

Comment 5 Kentaro Hara 2012-02-08 23:14:11 PST

Created attachment 126244 [details]
rebased patch for commit

Comment 6 Kentaro Hara 2012-02-08 23:20:46 PST

Committed r107186: <http://trac.webkit.org/changeset/107186>

Comment 7 Kentaro Hara 2012-02-09 03:21:03 PST

I rolled out the patch due to dependency with r107182. I'll re-land it after the patch for bug 78200 is landed successfully.

Comment 8 Kentaro Hara 2012-02-09 16:56:57 PST

Committed r107312: <http://trac.webkit.org/changeset/107312>