Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1348)

Unified Diff: chrome/browser/importer/profile_writer.h

Issue 2305813002: Remove ScopedVector from importer. (Closed)
Patch Set: pkasting Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/importer/in_process_importer_bridge.cc ('k') | chrome/browser/importer/profile_writer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/importer/profile_writer.h
diff --git a/chrome/browser/importer/profile_writer.h b/chrome/browser/importer/profile_writer.h
index cf2f1031c08463c62e2d056432d75c7bd47c0419..3913e09041c243e56b72b2b6e6fea1462c07c0f6 100644
--- a/chrome/browser/importer/profile_writer.h
+++ b/chrome/browser/importer/profile_writer.h
@@ -9,17 +9,16 @@
#include "base/macros.h"
#include "base/memory/ref_counted.h"
-#include "base/memory/scoped_vector.h"
#include "base/strings/string16.h"
#include "base/time/time.h"
#include "build/build_config.h"
#include "components/favicon_base/favicon_usage_data.h"
#include "components/history/core/browser/history_types.h"
+#include "components/search_engines/template_url_service.h"
#include "url/gurl.h"
struct ImportedBookmarkEntry;
class Profile;
-class TemplateURL;
namespace autofill {
struct PasswordForm;
@@ -78,16 +77,16 @@ class ProfileWriter : public base::RefCountedThreadSafe<ProfileWriter> {
virtual void AddFavicons(const favicon_base::FaviconUsageDataList& favicons);
- // Adds the TemplateURLs in |template_urls| to the local store. The local
- // store becomes the owner of the TemplateURLs. Some TemplateURLs in
- // |template_urls| may conflict (same keyword or same host name in the URL)
- // with existing TemplateURLs in the local store, in which case the existing
- // ones take precedence and the duplicates in |template_urls| are deleted.
- // If |unique_on_host_and_path| is true, a TemplateURL is only added if there
- // is not an existing TemplateURL that has a replaceable search url with the
- // same host+path combination.
- virtual void AddKeywords(ScopedVector<TemplateURL> template_urls,
- bool unique_on_host_and_path);
+ // Adds the TemplateURLs in |template_urls| to the local store.
+ // Some TemplateURLs in |template_urls| may conflict (same keyword or same
+ // host name in the URL) with existing TemplateURLs in the local store, in
+ // which case the existing ones take precedence and the duplicates in
+ // |template_urls| are deleted. If |unique_on_host_and_path| is true, a
+ // TemplateURL is only added if there is not an existing TemplateURL that has
+ // a replaceable search url with the same host+path combination.
+ virtual void AddKeywords(
+ TemplateURLService::OwnedTemplateURLVector template_urls,
+ bool unique_on_host_and_path);
// Adds the imported autofill entries to the autofill database.
virtual void AddAutofillFormDataEntries(
« no previous file with comments | « chrome/browser/importer/in_process_importer_bridge.cc ('k') | chrome/browser/importer/profile_writer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698