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

Unified Diff: chrome/browser/search_engines/template_url.cc

Issue 7566036: Implement SyncableServices in TemplateURLService. Add related unittests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Recommitting patch. Repaired memory leaks and adjusted unittests to reflect changes. Created 9 years, 4 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
Index: chrome/browser/search_engines/template_url.cc
===================================================================
--- chrome/browser/search_engines/template_url.cc (revision 97053)
+++ chrome/browser/search_engines/template_url.cc (working copy)
@@ -15,6 +15,7 @@
#include "chrome/browser/search_engines/search_engine_type.h"
#include "chrome/browser/search_engines/search_terms_data.h"
#include "chrome/browser/search_engines/template_url_service.h"
+#include "chrome/common/guid.h"
#include "chrome/common/url_constants.h"
#include "chrome/installer/util/google_update_settings.h"
#include "content/browser/user_metrics.h"
@@ -642,7 +643,8 @@
usage_count_(0),
search_engine_type_(SEARCH_ENGINE_OTHER),
logo_id_(kNoSearchEngineLogo),
- prepopulate_id_(0) {
+ prepopulate_id_(0),
+ sync_guid_(guid::GenerateGUID()) {
}
TemplateURL::~TemplateURL() {
@@ -730,10 +732,7 @@
void TemplateURL::SetPrepopulateId(int id) {
prepopulate_id_ = id;
- if (id > 0)
- SetTemplateURLRefsPrepopulated(true);
- else
- SetTemplateURLRefsPrepopulated(false);
+ SetTemplateURLRefsPrepopulated(id > 0);
}
void TemplateURL::InvalidateCachedValues() const {

Powered by Google App Engine
This is Rietveld 408576698