RefCounted types should not have public destructors, chrome/browser/ part 2

chrome/browser/sync/glue/data_type_controller.h

Index: chrome/browser/sync/glue/data_type_controller.h
diff --git a/chrome/browser/sync/glue/data_type_controller.h b/chrome/browser/sync/glue/data_type_controller.h
index 81f7c68b4ee02efe7b5a996f4ffb776a835ad085..2be6064ca679a53be1a74fa74cfbb6175d7efd62 100644
--- a/chrome/browser/sync/glue/data_type_controller.h
+++ b/chrome/browser/sync/glue/data_type_controller.h
@@ -105,18 +105,18 @@ class DataTypeController
       syncable::ModelType type) OVERRIDE;
 
  protected:
+  friend struct content::BrowserThread::DeleteOnThread<
+      content::BrowserThread::UI>;
+  friend class base::DeleteHelper<DataTypeController>;
+
+  virtual ~DataTypeController() {}
+
   // Handles the reporting of unrecoverable error. It records stuff in
   // UMA and reports to breakpad.
   // Virtual for testing purpose.
   virtual void RecordUnrecoverableError(
       const tracked_objects::Location& from_here,
       const std::string& message);
-
-  friend struct content::BrowserThread::DeleteOnThread<
-      content::BrowserThread::UI>;
-  friend class base::DeleteHelper<DataTypeController>;
-
-  virtual ~DataTypeController() {}
 };
 
 }  // namespace browser_sync