Importing certain bookmarks from firefox and HTML file as search engines.

chrome/utility/importer/bookmarks_file_importer.cc

 // Copyright 2013 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/bookmarks_file_importer.h"
 
 #include "base/bind.h"
 #include "chrome/common/importer/imported_bookmark_entry.h"
 #include "chrome/common/importer/imported_favicon_usage.h"
 #include "chrome/common/importer/importer_bridge.h"
@@ skipping 72 matching lines @@
     uint16 items,
     ImporterBridge* bridge) {
   // The only thing this importer can import is a bookmarks file, aka
   // "favorites".
   DCHECK_EQ(importer::FAVORITES, items);
 
   bridge->NotifyStarted();
   bridge->NotifyItemStarted(importer::FAVORITES);
 
   std::vector<ImportedBookmarkEntry> bookmarks;
+  std::vector<importer::URLKeywordInfo> url_keywords;
   std::vector<ImportedFaviconUsage> favicons;
 
   bookmark_html_reader::ImportBookmarksFile(
       base::Bind(IsImporterCancelled, base::Unretained(this)),
       base::Bind(internal::CanImportURL),
       source_profile.source_path,
       &bookmarks,
+      &url_keywords,
       &favicons);
 
   if (!bookmarks.empty() && !cancelled()) {
     base::string16 first_folder_name =
         bridge->GetLocalizedString(IDS_BOOKMARK_GROUP);
     bridge->AddBookmarks(bookmarks, first_folder_name);
   }
+  if (!url_keywords.empty() && !cancelled()) {

[Peter Kasting, 2014/10/07 23:17:39]

If we're supposed to check cancelled() here, are we supposed to check it in the
next conditional?  Should cancelled() result in some sort of early-return or
something?

[Tapu Ghose, 2014/10/12 00:58:21]

Removed check for cancelled().

+    bridge->SetKeywords(url_keywords, false);
+  }
   if (!favicons.empty())
     bridge->SetFavicons(favicons);
 
   bridge->NotifyItemEnded(importer::FAVORITES);
   bridge->NotifyEnded();
 }