wstrings: convert CppVariant and CppBoundClass to not use wstring

webkit/glue/cpp_variant.h

Index: webkit/glue/cpp_variant.h
diff --git a/webkit/glue/cpp_variant.h b/webkit/glue/cpp_variant.h
index 71b3166f89a9df3b30a382fefa670256d60d3393..82c43eec908629dbb3372c08b3120ce58f9ea3b0 100644
--- a/webkit/glue/cpp_variant.h
+++ b/webkit/glue/cpp_variant.h
@@ -22,6 +22,7 @@
 #include <vector>
 
 #include "base/basictypes.h"
+#include "base/string16.h"

[viettrungluu, 2010/12/03 02:00:16]

Why do you need this?

 #include "third_party/npapi/bindings/npruntime.h"
 
 class CppVariant : public NPVariant {
@@ -96,7 +97,7 @@ class CppVariant : public NPVariant {
   bool ToBoolean() const;
   // Returns a vector of strings for the specified argument. This is useful
   // for converting a JavaScript array of strings into a vector of strings.
-  std::vector<std::wstring> ToStringVector() const;
+  std::vector<std::string> ToStringVector() const;

[viettrungluu, 2010/12/03 02:00:16]

This sort of thing always worries me, since JS strings need not be real Unicode
(AFAIK, they're just strings of 16-bit numbers, really), but I think we handle
this "properly", whatever that means (since V8 stores strings as UTF-8(ish?)).

 
   // Invoke method of the given name on an object with the supplied arguments.
   // The first argument should be the object on which the method is to be