Remove the WCHAR_T_IS_UNSIGNED definition since it is not really useful....

Remove the WCHAR_T_IS_UNSIGNED definition since it is not really useful. Replace its use with a static_cast.

BUG=354405
NOTRY=true

Review URL: https://codereview.chromium.org/248843004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266058 0039d316-1c4b-4281-b951-d872f2087c98

base/strings/string_util_unittest.cc

@@ -1011,13 +1011,8 @@ TEST(StringUtilTest, LcpyTest) {
     EXPECT_EQ(1, dst[0]);
     EXPECT_EQ(2, dst[1]);
     EXPECT_EQ(7U, base::wcslcpy(wdst, L"abcdefg", 0));
-#if defined(WCHAR_T_IS_UNSIGNED)
-    EXPECT_EQ(1U, wdst[0]);
-    EXPECT_EQ(2U, wdst[1]);
-#else
-    EXPECT_EQ(1, wdst[0]);
-    EXPECT_EQ(2, wdst[1]);
-#endif
+    EXPECT_EQ(static_cast<wchar_t>(1), wdst[0]);
+    EXPECT_EQ(static_cast<wchar_t>(2), wdst[1]);
   }
 
   // Test the case were we _just_ competely fit including the null.

build/build_config.h

@@ -139,12 +139,6 @@
 #error Please add support for your compiler in build/build_config.h
 #endif
 
-#if defined(__ARMEL__) && !defined(OS_IOS)
-#define WCHAR_T_IS_UNSIGNED 1
-#elif defined(__MIPSEL__)
-#define WCHAR_T_IS_UNSIGNED 0
-#endif
-
 #if defined(OS_ANDROID)
 // The compiler thinks std::string::const_iterator and "const char*" are
 // equivalent types.