Fixes saving files that don't have valid extensions.

chrome/common/win_util.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.

[scherkus (not reviewing), 2009/04/17 19:36:41]

bump copyright year

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/common/win_util.h"
 
 #include <atlbase.h>
 #include <atlapp.h>
 #include <commdlg.h>
 #include <dwmapi.h>
 #include <shellapi.h>
@@ skipping 231 matching lines @@
       include_all_files = true;
       continue;
     }
 
     if (desc.empty()) {
       DCHECK(ext.find(L'.') != std::wstring::npos);
       std::wstring first_extension = ext.substr(ext.find(L'.'));
       size_t first_separator_index = first_extension.find(L';');
       if (first_separator_index != std::wstring::npos)
         first_extension = first_extension.substr(0, first_separator_index);
-      GetRegistryDescriptionFromExtension(first_extension, &desc);
+      if (!GetRegistryDescriptionFromExtension(first_extension, &desc)) {
+        // The extension doesn't exist in the registry. It's likely bogus, so
+        // just drop it.
+        include_all_files = true;
+        continue;
+      }
       if (desc.empty())
         desc = L"*." + first_extension;
     }
 
     result.append(desc.c_str(), desc.size()+1);  // Append NULL too.
     result.append(ext.c_str(), ext.size()+1);
   }
 
   if (include_all_files) {
     result.append(all_desc.c_str(), all_desc.size()+1);
@@ skipping 546 matching lines @@
 
 ChromeFont GetWindowTitleFont() {
   NONCLIENTMETRICS ncm;
   win_util::GetNonClientMetrics(&ncm);
   l10n_util::AdjustUIFont(&(ncm.lfCaptionFont));
   ScopedHFONT caption_font(CreateFontIndirect(&(ncm.lfCaptionFont)));
   return ChromeFont::CreateFont(caption_font);
 }
 
 }  // namespace win_util