[Sync] Replace uses of ObserverListThreadSafe with WeakHandles

chrome/browser/sync/syncable/directory_manager.h

 // 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.
 //
 // This used to do a lot of TLS-based management of multiple Directory objects.
 // We now can access Directory objects from any thread for general purpose
 // operations and we only ever have one Directory, so this class isn't doing
 // anything too fancy besides keeping calling and access conventions the same
 // for now.
 // TODO(timsteele): We can probably nuke this entire class and use raw
 // Directory objects everywhere.
 #ifndef CHROME_BROWSER_SYNC_SYNCABLE_DIRECTORY_MANAGER_H_
 #define CHROME_BROWSER_SYNC_SYNCABLE_DIRECTORY_MANAGER_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/file_path.h"
 #include "base/synchronization/lock.h"
 #include "chrome/browser/sync/syncable/dir_open_result.h"
 #include "chrome/browser/sync/syncable/syncable.h"
 #include "chrome/browser/sync/util/cryptographer.h"
+#include "chrome/browser/sync/util/weak_handle.h"
 
 namespace sync_api { class BaseTransaction; }
 namespace syncable { class BaseTransaction; }
 
 namespace syncable {
 
 class DirectoryChangeDelegate;
 
 class DirectoryManager {
  public:
 
   // root_path specifies where db is stored.
   explicit DirectoryManager(const FilePath& root_path);
   virtual ~DirectoryManager();
 
   static const FilePath GetSyncDataDatabaseFilename();
   const FilePath GetSyncDataDatabasePath() const;
 
   // Opens a directory.  Returns false on error.  Name parameter is
   // the the user's login, MUST already have been converted to a
   // common case.  Does not take ownership of |delegate|, which must
   // be non-NULL.  Starts sending events to |delegate| if the returned
   // result is true.  Note that events to |delegate| may be sent from
-  // *any* thread.
-  bool Open(const std::string& name, DirectoryChangeDelegate* delegate);
+  // *any* thread.  |transaction_observer| must be initialized.
+  bool Open(const std::string& name, DirectoryChangeDelegate* delegate,
+            const browser_sync::WeakHandle<TransactionObserver>&
+                transaction_observer);
 
   // Marks a directory as closed and stops sending events to the
   // delegate.  It might take a while until all the file handles and
   // resources are freed by other threads.
   void Close(const std::string& name);
 
   // Should be called at App exit.
   void FinalSaveChangesForAll();
 
   // Gets the list of currently open directory names.
   typedef std::vector<std::string> DirNames;
   void GetOpenDirectories(DirNames* result);
 
   // Wrappers for cryptographer() that enforce holding a transaction.
   // Note: the Cryptographer is NOT thread safe. It must only be accessed while
   // the transaction is still active. The Cryptographer's pointer should not be
   // stored separately.
   browser_sync::Cryptographer* GetCryptographer(
       const sync_api::BaseTransaction* trans) const { return cryptographer(); }
   browser_sync::Cryptographer* GetCryptographer(
       const syncable::BaseTransaction* trans) const { return cryptographer(); }
 
  protected:
   browser_sync::Cryptographer* cryptographer() const {
     return cryptographer_.get();
   }
 
-  DirOpenResult OpenImpl(const std::string& name, const FilePath& path,
-                         DirectoryChangeDelegate* delegate, bool* was_open);
+  DirOpenResult OpenImpl(
+      const std::string& name, const FilePath& path,
+      DirectoryChangeDelegate* delegate,
+      const browser_sync::WeakHandle<TransactionObserver>&
+          transaction_observer,
+      bool* was_open);
 
   // Helpers for friend class ScopedDirLookup:
   friend class ScopedDirLookup;
 
   const FilePath root_path_;
 
   // protects managed_directory_
   base::Lock lock_;
   Directory* managed_directory_;
 
@@ skipping 24 matching lines @@
   bool good_;
   // Ensure that the programmer checks good before using the ScopedDirLookup.
   // This member should can be removed if it ever shows up in profiling
   bool good_checked_;
   DirectoryManager* const dirman_;
 };
 
 }  // namespace syncable
 
 #endif  // CHROME_BROWSER_SYNC_SYNCABLE_DIRECTORY_MANAGER_H_