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

Unified Diff: chrome/browser/sync/api/sync_error.cc

Issue 7453014: [Sync] Refactor sync datatype error handling. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase and fix final unit test <3 c++ Created 9 years, 5 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/sync/api/sync_error.h ('k') | chrome/browser/sync/api/sync_error_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/api/sync_error.cc
diff --git a/chrome/browser/sync/api/sync_error.cc b/chrome/browser/sync/api/sync_error.cc
new file mode 100644
index 0000000000000000000000000000000000000000..c916baeb6e92b857744c2dcea2406eb9d90cb100
--- /dev/null
+++ b/chrome/browser/sync/api/sync_error.cc
@@ -0,0 +1,89 @@
+// Copyright (c) 2011 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/sync/api/sync_error.h"
+
+#include "base/logging.h"
+#include "base/tracked.h"
+
+SyncError::SyncError() {
+ Clear();
+}
+
+SyncError::SyncError(const tracked_objects::Location& location,
+ const std::string& message,
+ syncable::ModelType type) {
+ Init(location, message, type);
+ PrintLogError();
+}
+
+SyncError::SyncError(const SyncError& other) {
+ Copy(other);
+}
+
+SyncError& SyncError::operator=(const SyncError& other) {
+ if (this == &other) {
+ return *this;
+ }
+ Copy(other);
+ return *this;
+}
+
+void SyncError::Copy(const SyncError& other) {
+ if (other.IsSet()) {
+ Init(other.location(),
+ other.message(),
+ other.type());
+ } else {
+ Clear();
+ }
+}
+
+void SyncError::Clear() {
+ location_.reset();
+ message_ = std::string();
+ type_ = syncable::UNSPECIFIED;
+}
+
+void SyncError::Reset(const tracked_objects::Location& location,
+ const std::string& message,
+ syncable::ModelType type) {
+ Init(location, message, type);
+ PrintLogError();
+}
+
+void SyncError::Init(const tracked_objects::Location& location,
+ const std::string& message,
+ syncable::ModelType type) {
+ location_.reset(new tracked_objects::Location(location));
+ message_ = message;
+ type_ = type;
+}
+
+bool SyncError::IsSet() const {
+ return location_.get() != NULL;
+}
+
+const tracked_objects::Location& SyncError::location() const {
+ CHECK(IsSet());
+ return *location_;
+}
+
+const std::string& SyncError::message() const {
+ CHECK(IsSet());
+ return message_;
+}
+
+const syncable::ModelType SyncError::type() const {
+ CHECK(IsSet());
+ return type_;
+}
+
+void SyncError::PrintLogError() const {
+ LAZY_STREAM(logging::LogMessage(location_->file_name(),
+ location_->line_number(),
+ logging::LOG_ERROR).stream(),
+ LOG_IS_ON(ERROR))
+ << syncable::ModelTypeToString(type_) << " Sync Error: " << message_;
+}
« no previous file with comments | « chrome/browser/sync/api/sync_error.h ('k') | chrome/browser/sync/api/sync_error_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698