Refactoring the SysInfoProvider.

chrome/browser/chromeos/syslogs/syslogs_fetcher_factory.cc

+// 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/chromeos/syslogs/syslogs_fetcher_factory.h"
+
+#include "base/memory/singleton.h"
+#include "chrome/browser/chromeos/syslogs/commandline_fetcher.h"
+#include "chrome/browser/chromeos/syslogs/debugd_log_fetcher.h"
+#include "chrome/browser/chromeos/syslogs/memorydetails_fetcher.h"
+#include "chrome/browser/chromeos/syslogs/lsbrelease_fetcher.h"
+
+namespace chromeos {
+
+SysLogsFetcherFactory* SysLogsFetcherFactory::GetInstance() {
+  return Singleton<SysLogsFetcherFactory>::get();
+}
+
+SysLogsFetcher* SysLogsFetcherFactory::CreateFetcher() {
+  SysLogsDataSources data_sources;
+
+  // Debug Daemon data source.
+  data_sources.push_back(static_cast <SysLogsInterface*>(

[rkc, 2012/08/06 18:30:18]

no static cast needed.

[tudalex(Chromium), 2012/08/06 19:14:22]

Done.

+      new DebugDaemonLogFetcher()));
+
+  // Chrome data source.

[rkc, 2012/08/06 18:30:18]

nit: sources.

[tudalex(Chromium), 2012/08/06 19:14:22]

Done.

+  data_sources.push_back(new CommandLineFetcher());
+  data_sources.push_back(new LSBReleaseFetcher());
+  data_sources.push_back(new MemoryDetailsFetcher());
+  return new SysLogsFetcher(data_sources);
+}
+
+}  // namespace chromeos