Add comment for fix to issue 95619

chrome/browser/sync/profile_sync_service_harness.cc

Index: chrome/browser/sync/profile_sync_service_harness.cc
diff --git a/chrome/browser/sync/profile_sync_service_harness.cc b/chrome/browser/sync/profile_sync_service_harness.cc
index 1caa66d841eb83e336652f74078a69cad612db26..ffdf4544c3f01ca6ca3f261c12ea44980a3910ca 100644
--- a/chrome/browser/sync/profile_sync_service_harness.cc
+++ b/chrome/browser/sync/profile_sync_service_harness.cc
@@ -1005,6 +1005,8 @@ bool ProfileSyncServiceHarness::EnableEncryptionForType(
 
 bool ProfileSyncServiceHarness::WaitForTypeEncryption(
     syncable::ModelType type) {
+  // TODO(rlarocque): The correctness of this if condition depends on the

[akalin, 2011/09/22 19:13:47]

It's not clear what the TODO is.  Suggest:

// The correctness ... See crbug.com/95619.
// TODO(rlarocque): Figure out a less brittle way of detecting this.

[rlarocque, 2011/09/22 19:21:38]

Done.

+  // ordering of its sub-expressions.  See crbug.com/95619.
   if (IsTypeEncrypted(type) &&
       IsSynced() &&
       GetLastSessionSnapshot()->num_conflicting_updates == 0) {