Fix bug found by /analyze: incorrect return checking.

chrome/utility/importer/ie_importer_win.cc

 // Copyright (c) 2012 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 "chrome/utility/importer/ie_importer_win.h"
 
 #include <ole2.h>
 #include <intshcut.h>
 #include <shlobj.h>
 #include <urlhist.h>
@@ skipping 179 matching lines @@
 
     // Read the sort index of the current item.
     uint32 item_sort_index = 0;
     if (!BinaryRead(&item_sort_index, base_offset + kSortIndexOffset, blob))
       return false;
 
     // Read the file name from the ITEMIDLIST structure.
     LPCITEMIDLIST idlist = BinaryReadItemIDList(
       base_offset + kItemIDListOffset, item_size - kItemIDListOffset, blob);
     TCHAR item_filename[MAX_PATH];
-    if (!idlist || FAILED(SHGetPathFromIDList(idlist, item_filename)))
+    if (!idlist || !SHGetPathFromIDList(idlist, item_filename))
       return false;
     base::FilePath item_relative_path =
       path.Append(base::FilePath(item_filename).BaseName());
 
     // Record the retrieved information and go to the next item.
     sort_index->insert(std::make_pair(item_relative_path, item_sort_index));
     base_offset += item_size;
   }
   return true;
 }
@@ skipping 663 matching lines @@
   static int version = -1;
   if (version < 0) {
     wchar_t buffer[128];
     DWORD buffer_length = sizeof(buffer);
     base::win::RegKey reg_key(HKEY_LOCAL_MACHINE, kIEVersionKey, KEY_READ);
     LONG result = reg_key.ReadValue(L"Version", buffer, &buffer_length, NULL);
     version = ((result == ERROR_SUCCESS)? _wtoi(buffer) : 0);
   }
   return version;
 }