sync: Implement NonBlockingTypeProcessorCore

sync/engine/non_blocking_sync_common.cc

Index: sync/engine/non_blocking_sync_common.cc
diff --git a/sync/engine/non_blocking_sync_common.cc b/sync/engine/non_blocking_sync_common.cc
index c5a3656c8cb289f3bb734d8a938b10917f9e4c29..af9879fb49eb2d228a240d5bcbbf37de48cdbcff 100644
--- a/sync/engine/non_blocking_sync_common.cc
+++ b/sync/engine/non_blocking_sync_common.cc
@@ -6,25 +6,32 @@
 
 namespace syncer {
 
-DataTypeState::DataTypeState() {
+DataTypeState::DataTypeState() : next_client_id(0), initial_sync_done(false) {
 }
 
 DataTypeState::~DataTypeState() {
 }
 
-CommitRequestData::CommitRequestData() {
+CommitRequestData::CommitRequestData()
+    : sequence_number(0),
+      base_version(0),
+      deleted(false) {
 }
 
 CommitRequestData::~CommitRequestData() {
 }
 
-CommitResponseData::CommitResponseData() {
+CommitResponseData::CommitResponseData()
+    : sequence_number(0),
+      response_version() {

[Nicolas Zea, 2014/06/03 21:01:03]

0?

[rlarocque, 2014/06/03 21:14:34]

Done.

I'm slightly annoyed that this compiled without warnings.  I know it's
technically valid, but the odds of doing this intentionally are about as high as
for something like "if (foo = 123)"...

 }
 
 CommitResponseData::~CommitResponseData() {
 }
 
-UpdateResponseData::UpdateResponseData() {
+UpdateResponseData::UpdateResponseData()
+    : response_version(0),
+      deleted(false) {
 }
 
 UpdateResponseData::~UpdateResponseData() {