Refactoring the SysInfoProvider.

chrome/browser/chromeos/syslogs/syslogs_fetcher_factory.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 CHROME_BROWSER_CHROMEOS_SYSLOGS_SYSLOGS_FETCHER_FACTORY_H_
+#define CHROME_BROWSER_CHROMEOS_SYSLOGS_SYSLOGS_FETCHER_FACTORY_H_
+
+#include "base/memory/singleton.h"
+
+namespace chromeos {
+
+class SysLogsFetcher;
+
+// Provides a singleton SysLogsFetcherFactory that can be used to create
+// a SysLogsFetcher object that is used to fetch logs.
+class SysLogsFetcherFactory {
+ public:
+    static SysLogsFetcherFactory* GetInstance();
+    SysLogsFetcher* CreateFetcher();
+
+ private:
+  friend struct DefaultSingletonTraits<SysLogsFetcherFactory>;

[satorux1, 2012/08/08 22:01:03]

singleton should be avoided. can you make this a member of some class?

[tudalex(Chromium), 2012/08/09 02:30:24]

Talked to rkc@ and decided that we don't need this class anymore, moved it's
functionality into SysLogsFetcher constructor.

+
+  SysLogsFetcherFactory()  {}
+  ~SysLogsFetcherFactory() {}
+
+  DISALLOW_COPY_AND_ASSIGN(SysLogsFetcherFactory);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_SYSLOGS_SYSLOGS_FETCHER_FACTORY_H_
+