Use char instead of wchar_t for counter names....

Use char instead of wchar_t for counter names.

Counters should be ASCII anyway, and wchar_t is four bytes per character
on non-Windows platforms.

[Evan Martin, 2008-12-01 01:10:13]

Though the v8 contributor wiki says to send a patch to the mailing list, running
gcl seems to have picked up the appropriate mailing list.  So please let me know
if I should send this some other way.

(For those unfamiliar with this code, you should know that I've changed this
already on the Chrome side:
http://src.chromium.org/viewvc/chrome?view=rev&revision=5880
In particular, note that this file now has an adapter function to convert from
the wchar_t currently used by v8 to the ASCII expected by Chrome:
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/chromium_bridge_i...
)

[Dean McNamee, 2008-12-01 02:42:47]

LGTM

On 2008/12/01 01:10:13, Evan Martin wrote:
> Though the v8 contributor wiki says to send a patch to the mailing list,
running
> gcl seems to have picked up the appropriate mailing list.  So please let me
know
> if I should send this some other way.
> 
> (For those unfamiliar with this code, you should know that I've changed this
> already on the Chrome side:
> http://src.chromium.org/viewvc/chrome?view=rev&revision=5880
> In particular, note that this file now has an adapter function to convert from
> the wchar_t currently used by v8 to the ASCII expected by Chrome:
>
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/chromium_bridge_i...
> )

[Christian Plesner Hansen, 2008-12-01 07:24:33]

Thanks for changing this.  I've always been uneasy about us using wchar_t for
this (for anything, really) so it's great to get it cleaned up.

[Mads Ager (chromium), 2008-12-01 07:41:53]

I have committed this in v8 bleeding_edge revision 871.