[Sync] Remove SyncableServiceAdapter.

chrome/browser/sync/glue/generic_ui_data_type_controller.cc

Index: chrome/browser/sync/glue/generic_ui_data_type_controller.cc
diff --git a/chrome/browser/sync/glue/generic_ui_data_type_controller.cc b/chrome/browser/sync/glue/generic_ui_data_type_controller.cc
new file mode 100644
index 0000000000000000000000000000000000000000..ce72a30451835403e8ee52a46bec1cba544809d3
--- /dev/null
+++ b/chrome/browser/sync/glue/generic_ui_data_type_controller.cc
@@ -0,0 +1,31 @@
+// 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.
+
+#include "chrome/browser/sync/glue/generic_ui_data_type_controller.h"
+
+#include "chrome/browser/sync/syncable/model_type.h"
+
+using content::BrowserThread;
+
+namespace browser_sync {
+
+GenericUIDataTypeController::GenericUIDataTypeController(
+    syncable::ModelType type,
+    ProfileSyncComponentsFactory* profile_sync_factory,
+    Profile* profile,
+    ProfileSyncService* sync_service)
+    : UIDataTypeController(profile_sync_factory,
+                           profile,
+                           sync_service),
+      type_(type) {

[akalin, 2012/02/24 23:54:02]

any particular reason why you can't just make UIDTC take the type_ param too? 
It's a bit confusing to have two classes that are pretty similar...

[Nicolas Zea, 2012/03/01 20:50:26]

My main concern was that by having an implementation of the UIDTC always specify
the type (because the type() method in UIDTC remains purely virtual), you ensure
there's no possibility of using the wrong DTC for a specific type. I figured
it's less ambiguous when a UIDTC isn't standalone, and you have to specify some
implementation of it.

But, you're right, it would be simpler to have UIDTC just take the type as a
param and get rid of the Generic. Not sure my concerns are that valid. WDYT?

+}
+
+GenericUIDataTypeController::~GenericUIDataTypeController() {
+}
+
+syncable::ModelType GenericUIDataTypeController::type() const {
+  return type_;
+}
+
+}  // namespace browser_sync