| Index: chrome/browser/sync/glue/theme_model_associator.cc
|
| diff --git a/chrome/browser/sync/glue/theme_model_associator.cc b/chrome/browser/sync/glue/theme_model_associator.cc
|
| index e53b021620db3224d18f59d807d6dd5d099efcc9..5638badc37b79578abba3606c194b202edd3cc3b 100644
|
| --- a/chrome/browser/sync/glue/theme_model_associator.cc
|
| +++ b/chrome/browser/sync/glue/theme_model_associator.cc
|
| @@ -8,6 +8,7 @@
|
| #include "base/logging.h"
|
| #include "base/tracked.h"
|
| #include "base/utf_string_conversions.h"
|
| +#include "chrome/browser/sync/api/sync_error.h"
|
| #include "chrome/browser/sync/engine/syncapi.h"
|
| #include "chrome/browser/sync/glue/sync_backend_host.h"
|
| #include "chrome/browser/sync/glue/theme_util.h"
|
| @@ -35,11 +36,11 @@ ThemeModelAssociator::ThemeModelAssociator(
|
|
|
| ThemeModelAssociator::~ThemeModelAssociator() {}
|
|
|
| -bool ThemeModelAssociator::AssociateModels() {
|
| +bool ThemeModelAssociator::AssociateModels(SyncError* error) {
|
| sync_api::WriteTransaction trans(FROM_HERE, sync_service_->GetUserShare());
|
| sync_api::ReadNode root(&trans);
|
| if (!root.InitByTagLookup(kThemesTag)) {
|
| - LOG(ERROR) << kNoThemesFolderError;
|
| + error->Reset(FROM_HERE, kNoThemesFolderError, model_type());
|
| return false;
|
| }
|
|
|
| @@ -62,7 +63,9 @@ bool ThemeModelAssociator::AssociateModels() {
|
| sync_api::WriteNode node(&trans);
|
| if (!node.InitUniqueByCreation(syncable::THEMES, root,
|
| kCurrentThemeClientTag)) {
|
| - LOG(ERROR) << "Could not create current theme node.";
|
| + error->Reset(FROM_HERE,
|
| + "Could not create current theme node.",
|
| + model_type());
|
| return false;
|
| }
|
| node.SetIsFolder(false);
|
| @@ -74,7 +77,7 @@ bool ThemeModelAssociator::AssociateModels() {
|
| return true;
|
| }
|
|
|
| -bool ThemeModelAssociator::DisassociateModels() {
|
| +bool ThemeModelAssociator::DisassociateModels(SyncError* error) {
|
| // We don't maintain any association state, so nothing to do.
|
| return true;
|
| }
|
|
|