Update some comments/nits following r7486.

base/json_reader.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 "base/json_reader.h"
 
 #include "base/float_util.h"
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "base/values.h"
@@ skipping 118 matching lines @@
   }
 
   // The conversion from UTF8 to wstring removes null bytes for us
   // (a good thing).
   std::wstring json_wide(UTF8ToWide(json));
   start_pos_ = json_wide.c_str();
 
   // When the input JSON string starts with a UTF-8 Byte-Order-Mark
   // (0xEF, 0xBB, 0xBF), the UTF8ToWide() function converts it to a Unicode
   // BOM (U+FEFF). To avoid the JSONReader::BuildValue() function from
-  // mis-treating a Unicode BOM as an invalid character and returning false,
+  // mis-treating a Unicode BOM as an invalid character and returning NULL,
   // skip a converted Unicode BOM if it exists.
   if (!json_wide.empty() && start_pos_[0] == 0xFEFF) {
     ++start_pos_;
   }
 
   json_pos_ = start_pos_;
   allow_trailing_comma_ = allow_trailing_comma;
   stack_depth_ = 0;
   error_message_.clear();
 
@@ skipping 60 matching lines @@
       break;
 
     case Token::ARRAY_BEGIN:
       {
         json_pos_ += token.length;
         token = ParseToken();
 
         node.reset(new ListValue());
         while (token.type != Token::ARRAY_END) {
           Value* array_node = BuildValue(false);
-          if (!array_node) {
+          if (!array_node)
             return NULL;
-          }
           static_cast<ListValue*>(node.get())->Append(array_node);
 
           // After a list value, we expect a comma or the end of the list.
           token = ParseToken();
           if (token.type == Token::LIST_SEPARATOR) {
             json_pos_ += token.length;
             token = ParseToken();
             // Trailing commas are invalid according to the JSON RFC, but some
             // consumers need the parsing leniency, so handle accordingly.
             if (token.type == Token::ARRAY_END) {
@@ skipping 398 matching lines @@
     if (*pos == '\n') {
       ++line_number;
       column_number = 1;
     } else {
       ++column_number;
     }
   }
 
   error_message_ = FormatErrorMessage(line_number, column_number, description);
 }