Sync: Add DeviceInfo's ChangeProcessor

sync/internal_api/public/base_transaction.h

 // 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.
 
 #ifndef SYNC_INTERNAL_API_PUBLIC_BASE_TRANSACTION_H_
 #define SYNC_INTERNAL_API_PUBLIC_BASE_TRANSACTION_H_
 
 #include "sync/internal_api/public/user_share.h"
 
 #include "sync/internal_api/public/base/model_type.h"
@@ skipping 14 matching lines @@
 // whose construction requires an explicit syncable::Directory, a sync
 // API BaseTransaction is created from a UserShare object.
 class BaseTransaction {
  public:
   // Provide access to the underlying syncable objects from BaseNode.
   virtual syncable::BaseTransaction* GetWrappedTrans() const = 0;
   Cryptographer* GetCryptographer() const;
   ModelTypeSet GetEncryptedTypes() const;
 
   syncable::Directory* GetDirectory() const {
-    return directory_;
+    if (!user_share_) {

[Nicolas Zea, 2012/11/16 19:42:27]

remove curly braces?

[rlarocque, 2012/11/17 00:07:21]

I'd prefer not to.  I believe the style guide does not specify either way, as
longs as the style is consistent within the file.

+      return NULL;
+    } else {
+      return user_share_->directory.get();
+    }
+  }
+
+  UserShare* GetUserShare() const {
+    return user_share_;
   }
 
  protected:
   explicit BaseTransaction(UserShare* share);
   virtual ~BaseTransaction();
 
-  BaseTransaction() : directory_(NULL) { }
+  BaseTransaction() : user_share_(NULL) { }
 
  private:
-  syncable::Directory* directory_;
+  UserShare* user_share_;
 
   DISALLOW_COPY_AND_ASSIGN(BaseTransaction);
 };
 
 }  // namespace syncer
 
 #endif  // SYNC_INTERNAL_API_PUBLIC_BASE_TRANSACTION_H_