Fixing potential memory leak in BookmarkCodec::DecodeNode....

chrome/browser/bookmarks/bookmark_codec.h

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// Copyright (c) 2009 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.
 
 // BookmarkCodec is responsible for encoding and decoding the BookmarkModel
 // into JSON values. The encoded values are written to disk via the
 // BookmarkService.
 
 #ifndef CHROME_BROWSER_BOOKMARKS_BOOKMARK_CODEC_H_
 #define CHROME_BROWSER_BOOKMARKS_BOOKMARK_CODEC_H_
 
@@ skipping 90 matching lines @@
                       BookmarkNode* parent);
 
   // Reassigns bookmark IDs for all nodes.
   void ReassignIDs(BookmarkNode* bb_node, BookmarkNode* other_node);
 
   // Helper to recursively reassign IDs.
   void ReassignIDsHelper(BookmarkNode* node);
 
   // Decodes the supplied node from the supplied value. Child nodes are
   // created appropriately by way of DecodeChildren. If node is NULL a new
-  // node is created and added to parent, otherwise node is used.
+  // node is created and added to parent (parent must then be non-NULL),
+  // otherwise node is used.
   bool DecodeNode(const DictionaryValue& value,
                   BookmarkNode* parent,
                   BookmarkNode* node);
 
   // Updates the check-sum with the given string.
   void UpdateChecksum(const std::string& str);
   void UpdateChecksum(const std::wstring& str);
 
   // Updates the check-sum with the given contents of URL/folder bookmark node.
   // NOTE: These functions take in individual properties of a bookmark node
@@ skipping 23 matching lines @@
   std::string computed_checksum_;
   std::string stored_checksum_;
 
   // Maximum ID assigned when decoding data.
   int64 maximum_id_;
 
   DISALLOW_COPY_AND_ASSIGN(BookmarkCodec);
 };
 
 #endif  // CHROME_BROWSER_BOOKMARKS_BOOKMARK_CODEC_H_