[Sync] Add plumbing of context from client to server

chrome/browser/sync/glue/generic_change_processor.cc

 // Copyright (c) 2012 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/glue/generic_change_processor.h"
 
 #include "base/location.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "content/public/browser/browser_thread.h"
 #include "sync/api/sync_change.h"
 #include "sync/api/sync_error.h"
 #include "sync/api/syncable_service.h"
 #include "sync/internal_api/public/base_node.h"
 #include "sync/internal_api/public/change_record.h"
 #include "sync/internal_api/public/read_node.h"
 #include "sync/internal_api/public/read_transaction.h"
 #include "sync/internal_api/public/util/unrecoverable_error_handler.h"
 #include "sync/internal_api/public/write_node.h"
 #include "sync/internal_api/public/write_transaction.h"
+#include "sync/syncable/directory.h"
 #include "sync/syncable/entry.h"  // TODO(tim): Bug 123674.
 
 using content::BrowserThread;
 
 namespace browser_sync {
 
 namespace {
 
 void SetNodeSpecifics(const sync_pb::EntitySpecifics& entity_specifics,
                       syncer::WriteNode* write_node) {
@@ skipping 115 matching lines @@
     syncer::ModelType type) const {
   // This is slow / memory intensive.  Should be used sparingly by datatypes.
   syncer::SyncDataList data;
   GetAllSyncDataReturnError(type, &data);
   return data;
 }
 
 syncer::SyncError GenericChangeProcessor::UpdateDataTypeContext(
     syncer::ModelType type,
     const std::string& context) {
-  NOTIMPLEMENTED();
+  syncer::WriteTransaction trans(FROM_HERE, share_handle());
+  sync_pb::DataTypeContext context_proto;
+  trans.GetDirectory()->GetDataTypeContext(type, &context_proto);
+
+  if (context_proto.context() == context)
+    return syncer::SyncError();  // Context already matches.

[rlarocque, 2014/04/03 17:31:58]

Why is this an error?  It seems harmless.

[Nicolas Zea, 2014/04/03 22:56:47]

default-initialized SyncError aren't errors.

+
+  if (context_proto.type() == 0)
+    context_proto.set_type(syncer::GetSpecificsFieldNumberFromModelType(type));
+  DCHECK_EQ(syncer::GetSpecificsFieldNumberFromModelType(type),
+            context_proto.type());
+  DCHECK_GE(context_proto.version(), 0);
+  context_proto.set_version(context_proto.version() + 1);
+  context_proto.set_context(context);
+  trans.GetDirectory()->SetDataTypeContext(type, context_proto);
   return syncer::SyncError();
 }
 
 syncer::SyncError GenericChangeProcessor::GetAllSyncDataReturnError(
     syncer::ModelType type,
     syncer::SyncDataList* current_sync_data) const {
   DCHECK(CalledOnValidThread());
   std::string type_name = syncer::ModelTypeToString(type);
   syncer::ReadTransaction trans(FROM_HERE, share_handle());
   syncer::ReadNode root(&trans);
@@ skipping 436 matching lines @@
 void GenericChangeProcessor::StartImpl(Profile* profile) {
   DCHECK(CalledOnValidThread());
 }
 
 syncer::UserShare* GenericChangeProcessor::share_handle() const {
   DCHECK(CalledOnValidThread());
   return share_handle_;
 }
 
 }  // namespace browser_sync