NOT FOR COMMIT: Add DeviceInfo type and ChangeProcessor

chrome/browser/sync/glue/DEPS

Index: chrome/browser/sync/glue/DEPS
diff --git a/chrome/browser/sync/glue/DEPS b/chrome/browser/sync/glue/DEPS
index e61591fc7b2b0424e90ae6e608b3f23304bfdd60..dc6557540dcab3de4f8d5f3677a25c5f74a4467a 100644
--- a/chrome/browser/sync/glue/DEPS
+++ b/chrome/browser/sync/glue/DEPS
@@ -5,6 +5,7 @@ include_rules = [
 
   "+sync/internal_api/public",
   "+sync/util",
+  "+sync/test",

[Nicolas Zea, 2012/09/14 19:15:08]

Could the mock transaction observer (at least the header) be moved to
sync/internal_api/public/test? 

[rlarocque, 2012/09/15 01:36:37]

I agree that it should be moved somewhere.  I tentatively put it in sync/test
because the other test transaction observer, NullTransactionObserver, lives
there.  

I don't think that sync/internal_api/public_test is the right place to put it,
though.  The TransactionObserver interface is in syncable, not internal_api. 
I'm also worried about the implications the move would have on linking/symbols.

[Nicolas Zea, 2012/09/17 17:54:00]

One solution is to just have the header file not expose the observer
implementation, and instead have the only transactions_observed() method. You'd
then have a a transaction observer object (which you forward declare in the
header), whose implementation is in the cc file. Similar to SBH::Core.

I think in that scenario it would also be appropriate to rename the main class
to SyncNudgeObserver or something, since the fact that you're using a
transaction observer is an implementation detail.

 
   "+jingle/notifier/base",