IsStringUTF8 unittest and enforcing UTF-8 in JSON deserialization

net/base/net_util.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <algorithm>
 #include <unicode/ucnv.h>
 #include <unicode/uidna.h>
 #include <unicode/ulocdata.h>
 #include <unicode/uniset.h>
 #include <unicode/uscript.h>
@@ skipping 219 matching lines @@
 }
 
 bool DecodeWord(const std::string& encoded_word,
                 bool *is_rfc2047,
                 std::string* output) {
   // TODO(jungshik) : Revisit this later. Do we want to pass through non-ASCII
   // strings which can be mozibake?  WinHTTP converts a raw 8bit string
   // UTF-16 assuming it's in the OS default encoding.
   if (!IsStringASCII(encoded_word)) {
     // Try falling back to the NativeMB encoding if the raw input is not UTF-8.
-    if (IsStringUTF8(encoded_word.c_str())) {
+    if (IsStringUTF8(encoded_word)) {
       *output = encoded_word;
     } else {
       *output = WideToUTF8(base::SysNativeMBToWide(encoded_word));
     }
     *is_rfc2047 = false;
     return true;
   }
 
   // RFC 2047 : one of encoding methods supported by Firefox and relatively
   // widely used by web servers.
@@ skipping 70 matching lines @@
   }
 
   // We're not handling 'especial' characters quoted with '\', but
   // it should be Ok because we're not an email client but a
   // web browser.
 
   // What IE6/7 does: %-escaped UTF-8. We could extend this to
   // support a rudimentary form of RFC 2231 with charset label, but
   // it'd gain us little in terms of compatibility.
   tmp = UnescapeURLComponent(encoded_word, UnescapeRule::SPACES);
-  if (IsStringUTF8(tmp.c_str())) {
+  if (IsStringUTF8(tmp)) {
     output->swap(tmp);
     return true;
     // We can try either the OS default charset or 'origin charset' here,
     // As far as I can tell, IE does not support it. However, I've seen
     // web servers emit %-escaped string in a legacy encoding (usually
     // origin charset).
     // TODO(jungshik) : Test IE further and consider adding a fallback here.
   }
   return false;
 }
@@ skipping 565 matching lines @@
   for (int i = 0; i < array_size; i++) {
     if (kAllowedFtpPorts[i] == port) {
         return true;
     }
   }
   // Port not explicitly allowed by FTP, so return the default restrictions.
   return IsPortAllowedByDefault(port);
 }
 
 }  // namespace net