[USS] Implement SharedModelTypeProcessor::GetAllNodes for USS

components/sync/api/entity_data.cc

Index: components/sync/api/entity_data.cc
diff --git a/components/sync/api/entity_data.cc b/components/sync/api/entity_data.cc
index 59500fa3324b41cca4ce0b967f27ed811b73632c..9e518ae1839db08eaa13bbee6a59208a82f583fd 100644
--- a/components/sync/api/entity_data.cc
+++ b/components/sync/api/entity_data.cc
@@ -7,9 +7,21 @@
 #include <algorithm>
 
 #include "base/logging.h"
+#include "base/strings/string_number_conversions.h"
+#include "base/strings/string_util.h"
+#include "components/sync/base/time.h"
+#include "components/sync/base/unique_position.h"
+#include "components/sync/protocol/proto_value_conversions.h"
 
 namespace syncer_v2 {
 
+namespace {

[pavely, 2016/09/14 17:15:30]

nit: new line after "namespace syncer_v2 {" and before "}  // namespace"

[maxbogue, 2016/09/14 17:58:02]

Since there's already a newline after namespace syncer_v2, I assume pavel means
after "namespace {"

[Gang Wu, 2016/09/15 17:12:47]

Done.

[Gang Wu, 2016/09/15 17:12:47]

Done.

+std::string UniquePositionToString(
+    const sync_pb::UniquePosition& unique_position) {
+  return syncer::UniquePosition::FromProto(unique_position).ToDebugString();
+}
+}  // namespace
+
 EntityData::EntityData() {}
 EntityData::~EntityData() {}
 
@@ -33,6 +45,24 @@ EntityDataPtr EntityData::PassToPtr() {
   return target;
 }
 
+#define ADD_TO_DICT(dict, value, transform) \

[maxbogue, 2016/09/14 17:58:02]

Despite this being how proto_value_conversions does it, I still think it would
be more readable without the macro.

[Gang Wu, 2016/09/15 17:12:47]

Acknowledged.

+  dict->SetString(base::ToUpperASCII(#value), transform(value));
+
+std::unique_ptr<base::DictionaryValue> EntityData::ToDictionaryValue() {
+  std::unique_ptr<base::DictionaryValue> dict =
+      syncer::EntitySpecificsToValue(specifics);
+  ADD_TO_DICT(dict, id, );
+  ADD_TO_DICT(dict, client_tag_hash, );
+  ADD_TO_DICT(dict, non_unique_name, );
+  ADD_TO_DICT(dict, parent_id, );
+  ADD_TO_DICT(dict, creation_time, syncer::GetTimeDebugString);
+  ADD_TO_DICT(dict, modification_time, syncer::GetTimeDebugString);
+  ADD_TO_DICT(dict, unique_position, UniquePositionToString);
+  return dict;
+}
+
+#undef ADD_TO_DICT
+
 void EntityDataTraits::SwapValue(EntityData* dest, EntityData* src) {
   dest->Swap(src);
 }