[Sync] Remove ModelError::IsSet() in favor of base::Optional.

components/sync/model/model_error.cc

Index: components/sync/model/model_error.cc
diff --git a/components/sync/model/model_error.cc b/components/sync/model/model_error.cc
index 6ae66101d8ba9a10276d368fba12559882c40165..8b65536a90badd038d7515a9822467f3cf4ca511 100644
--- a/components/sync/model/model_error.cc
+++ b/components/sync/model/model_error.cc
@@ -6,25 +6,17 @@
 
 namespace syncer {
 
-ModelError::ModelError() : is_set_(false) {}
-
 ModelError::ModelError(const tracked_objects::Location& location,
                        const std::string& message)
-    : is_set_(true), location_(location), message_(message) {}
+    : location_(location), message_(message) {}
 
 ModelError::~ModelError() = default;
 
-bool ModelError::IsSet() const {
-  return is_set_;
-}
-
 const tracked_objects::Location& ModelError::location() const {
-  DCHECK(IsSet());
   return location_;
 }
 
 const std::string& ModelError::message() const {
-  DCHECK(IsSet());
   return message_;
 }