Implement new SyncAPI and convert Preferences to it.

chrome/chrome_browser.gypi

 # 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.
 
 {
   'targets': [
     {
       'target_name': 'browser',
       'type': '<(library)',
       'msvs_guid': '5BF908A7-68FB-4A4B-99E3-8C749F1FE4EA',
@@ skipping 1933 matching lines @@
         'browser/sync/glue/password_model_worker.cc',
         'browser/sync/glue/password_model_worker.h',
         'browser/sync/glue/preference_data_type_controller.cc',
         'browser/sync/glue/preference_data_type_controller.h',
         'browser/sync/glue/session_change_processor.cc',
         'browser/sync/glue/session_change_processor.h',
         'browser/sync/glue/session_data_type_controller.cc',
         'browser/sync/glue/session_data_type_controller.h',
         'browser/sync/glue/session_model_associator.cc',
         'browser/sync/glue/session_model_associator.h',
+        'browser/sync/glue/syncable_service_adapter.cc',
+        'browser/sync/glue/syncable_service_adapter.h',        
         'browser/sync/glue/sync_backend_host.cc',
         'browser/sync/glue/sync_backend_host.h',
         'browser/sync/glue/theme_change_processor.cc',
         'browser/sync/glue/theme_change_processor.h',
         'browser/sync/glue/theme_data_type_controller.cc',
         'browser/sync/glue/theme_data_type_controller.h',
         'browser/sync/glue/theme_model_associator.cc',
         'browser/sync/glue/theme_model_associator.h',
         'browser/sync/glue/theme_util.cc',
         'browser/sync/glue/theme_util.h',
         'browser/sync/glue/typed_url_change_processor.cc',
         'browser/sync/glue/typed_url_change_processor.h',
         'browser/sync/glue/typed_url_data_type_controller.cc',
         'browser/sync/glue/typed_url_data_type_controller.h',
         'browser/sync/glue/typed_url_model_associator.cc',
         'browser/sync/glue/typed_url_model_associator.h',
         'browser/sync/glue/ui_model_worker.cc',
         'browser/sync/glue/ui_model_worker.h',
         'browser/sync/backend_migrator.cc',
         'browser/sync/backend_migrator.h',
         'browser/sync/profile_sync_factory.h',
         'browser/sync/profile_sync_factory_impl.cc',
         'browser/sync/profile_sync_factory_impl.h',
         'browser/sync/profile_sync_service.cc',
         'browser/sync/profile_sync_service.h',
         'browser/sync/profile_sync_service_harness.cc',
         'browser/sync/profile_sync_service_harness.h',
         'browser/sync/profile_sync_service_observer.h',
+        'browser/sync/api/syncable_service.cc',
+        'browser/sync/api/syncable_service.h',
+        'browser/sync/api/sync_data.h',
+        'browser/sync/api/sync_data.cc',

[akalin, 2011/05/19 00:58:45]

can you move the api stuff to its own target?  (if it's not too hairy)

[Nicolas Zea, 2011/05/19 21:17:45]

I suppose I could, but what would be the benefit? This is definitely a part of
the browser in my view.

[akalin, 2011/05/20 00:08:05]

Well by that logic, we should just have everything in one 'browser' target,
since we're writing a browser. :-)

Currently the sync code is a hairball of tangled dependencies that is nearly
impossible to modularize.  Given that this is mostly new code, I'd like to not
add to the hairball, although this might end up being infeasible (if the api
stuff depends back on browser).

[Nicolas Zea, 2011/05/20 02:00:28]

Discussed offline

+        'browser/sync/api/sync_event.h',
+        'browser/sync/api/sync_event.cc',
+        'browser/sync/api/sync_event_processor.h',
         'browser/sync/signin_manager.cc',
         'browser/sync/signin_manager.h',
-        'browser/sync/syncable_service.cc',
-        'browser/sync/syncable_service.h',
         'browser/sync/sync_setup_flow.cc',
         'browser/sync/sync_setup_flow.h',
         'browser/sync/sync_setup_flow_handler.h',
         'browser/sync/sync_setup_wizard.cc',
         'browser/sync/sync_setup_wizard.h',
         'browser/sync/sync_ui_util.cc',
         'browser/sync/sync_ui_util.h',
         'browser/sync/sync_ui_util_mac.h',
         'browser/sync/sync_ui_util_mac.mm',
         'browser/tab_closeable_state_watcher.cc',
@@ skipping 2565 matching lines @@
       'hard_dependency': 1,
     },
   ],
 }
 
 # Local Variables:
 # tab-width:2
 # indent-tabs-mode:nil
 # End:
 # vim: set expandtab tabstop=2 shiftwidth=2: