Add comment for fix to issue 95619

chrome/browser/sync/profile_sync_service_harness.cc

 // Copyright (c) 2011 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 "chrome/browser/sync/profile_sync_service_harness.h"
 
 #include <stddef.h>
 #include <algorithm>
 #include <iterator>
 #include <ostream>
@@ skipping 987 matching lines @@
       (syncable::MODEL_TYPE_COUNT - syncable::FIRST_REAL_MODEL_TYPE));
   service_->OnUserChoseDatatypes(sync_everything,
                                  synced_datatypes);
 
   // Wait some time to let the enryption finish.
   return WaitForTypeEncryption(type);
 }
 
 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) {
     // Encryption is already complete for |type|; do not wait.
     return true;
   }
 
   std::string reason = "Waiting for encryption.";
   wait_state_ = WAITING_FOR_ENCRYPTION;
   waiting_for_encryption_type_ = type;
@@ skipping 24 matching lines @@
   return (synced_types.count(type) != 0);
 }
 
 std::string ProfileSyncServiceHarness::GetServiceStatus() {
   DictionaryValue value;
   sync_ui_util::ConstructAboutInformation(service_, &value);
   std::string service_status;
   base::JSONWriter::Write(&value, true, &service_status);
   return service_status;
 }