[Sync] Add location parameter to SyncChange in Sync API.

sync/api/sync_change.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 "sync/api/sync_change.h"
 
 #include <ostream>
 
 namespace syncer {
 
 SyncChange::SyncChange() : change_type_(ACTION_INVALID) {
 }
 
-SyncChange::SyncChange(SyncChangeType change_type, const SyncData& sync_data)
-    : change_type_(change_type),
+SyncChange::SyncChange(
+    const tracked_objects::Location& from_here,
+    SyncChangeType change_type,
+    const SyncData& sync_data)
+    : location_(from_here),
+      change_type_(change_type),
       sync_data_(sync_data) {
   DCHECK(IsValid());
 }
 
 SyncChange::~SyncChange() {}
 
 bool SyncChange::IsValid() const {
   if (change_type_ == ACTION_INVALID || !sync_data_.IsValid())
     return false;
 
@@ skipping 33 matching lines @@
       return "ACTION_UPDATE";
     case ACTION_DELETE:
       return "ACTION_DELETE";
     default:
       NOTREACHED();
   }
   return std::string();
 }
 
 std::string SyncChange::ToString() const {
-  return "{ changeType: " + ChangeTypeToString(change_type_) +
-      ", syncData: " + sync_data_.ToString() + "}";
+  return "{ " + location_.ToString() + ", changeType: " +

[rlarocque, 2012/07/10 22:09:54]

Should there be a "location: " prefix string here?

[Nicolas Zea, 2012/07/10 22:13:35]

I think locations are pretty recognizable enough to not merit a prefix
(particularly since they have their own special format).

+      ChangeTypeToString(change_type_) + ", syncData: " +
+      sync_data_.ToString() + "}";
 }
 
 void PrintTo(const SyncChange& sync_change, std::ostream* os) {
   *os << sync_change.ToString();
 }
 
 }  // namespace syncer