Sync/Valgrind: Add gmock printers for SyncChange, SyncData, SyncError.

chrome/browser/sync/api/sync_change.h

Index: chrome/browser/sync/api/sync_change.h
diff --git a/chrome/browser/sync/api/sync_change.h b/chrome/browser/sync/api/sync_change.h
index f2bc40e05070dacd49d3ce8e6c8b9b35ddc1ebce..9ca5b7c8f7201f8fce7f9962e36cf8c1042c8a50 100644
--- a/chrome/browser/sync/api/sync_change.h
+++ b/chrome/browser/sync/api/sync_change.h
@@ -6,6 +6,7 @@
 #define CHROME_BROWSER_SYNC_API_SYNC_CHANGE_H_
 #pragma once
 
+#include <iosfwd>
 #include <string>
 #include <vector>
 
@@ -56,4 +57,7 @@ class SyncChange {
   SyncData sync_data_;
 };
 
+// Printing operator - helps gtest produce readable error messages.
+std::ostream& operator<<(std::ostream& os, const SyncChange& sync_change);

[akalin, 2011/09/21 23:12:58]

This is against Google style.  I'd prefer to add 'const std::string ToString()
const' member functions to SyncChange etc. and also define PrintTo() for gtest
as described in http://code.google.com/p/googletest/wiki/AdvancedGuide .

[James Hawkins, 2011/09/22 01:15:26]

Done.

+
 #endif  // CHROME_BROWSER_SYNC_API_SYNC_CHANGE_H_