Fixes bug in importer where we could set the default search provider...

chrome/browser/importer/firefox_importer_utils.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 "chrome/browser/importer/firefox_importer_utils.h"
 
 #include <shlobj.h>
 
 #include "base/file_util.h"
 #include "base/logging.h"
@@ skipping 310 matching lines @@
     const std::wstring& profile_path) {
   // The default search engine is contained in the file prefs.js found in the
   // profile directory.
   // It is the "browser.search.selectedEngine" property.
   if (search_engines.empty())
     return -1;
 
   std::wstring default_se_name = UTF8ToWide(
       ReadPrefsJsValue(profile_path, "browser.search.selectedEngine"));
 
+  if (default_se_name.empty()) {
+    // browser.search.selectedEngine does not exist if the user has not changed
+    // from the default (or has selected the default).
+    // TODO: should fallback to 'browser.search.defaultengine' if selectedEngine
+    // is empty.
+    return -1;
+  }
+
   int default_se_index = -1;
   for (std::vector<TemplateURL*>::const_iterator iter = search_engines.begin();
        iter != search_engines.end(); ++iter) {
     if (default_se_name == (*iter)->short_name()) {
       default_se_index = static_cast<int>(iter - search_engines.begin());
       break;
     }
   }
   if (default_se_index == -1) {
     NOTREACHED() <<
@@ skipping 377 matching lines @@
       // Version 3 has an extra line for further use.
       if (version == 3) {
         ++begin;
       }
 
       forms->push_back(form);
     }
   }
 }