RESOLVED FIXED Bug 78108
Rename [ConvertNullStringTo=] to [TreatReturnedNullStringAs=] 
https://bugs.webkit.org/show_bug.cgi?id=78108
Summary Rename [ConvertNullStringTo=] to [TreatReturnedNullStringAs=]
Kentaro Hara
Reported 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].
Attachments
Patch (43.86 KB, patch)
2012-02-08 05:50 PST, Kentaro Hara 		abarth: review+
webkit.review.bot: commit-queue-
DetailsFormatted DiffDiff
rebased patch for commit (44.06 KB, patch)
2012-02-08 16:04 PST, Kentaro Hara 		haraken: commit-queue-
DetailsFormatted DiffDiff
rebased patch for commit (43.90 KB, patch)
2012-02-08 23:14 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kentaro Hara
Comment 1 2012-02-08 05:50:13 PST
Created attachment 126067 [details] Patch
WebKit Review Bot
Comment 2 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
Kentaro Hara
Comment 3 2012-02-08 16:04:02 PST
Created attachment 126173 [details] rebased patch for commit
WebKit Review Bot
Comment 4 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
Kentaro Hara
Comment 5 2012-02-08 23:14:11 PST
Created attachment 126244 [details] rebased patch for commit
Kentaro Hara
Comment 6 2012-02-08 23:20:46 PST
Committed r107186: <http://trac.webkit.org/changeset/107186>
Kentaro Hara
Comment 7 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.
Kentaro Hara
Comment 8 2012-02-09 16:56:57 PST
Committed r107312: <http://trac.webkit.org/changeset/107312>
Note You need to log in before you can comment on or make changes to this bug.