Use scrubbed logs for sending with feedback reports.

chrome/browser/chromeos/system_logs/about_system_logs_fetcher.h

+// Copyright 2013 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_SYSTEM_LOGS_ABOUT_SYSTEM_LOGS_FETCHER_H_
+#define CHROME_BROWSER_CHROMEOS_SYSTEM_LOGS_ABOUT_SYSTEM_LOGS_FETCHER_H_
+
+#include "chrome/browser/chromeos/system_logs/system_logs_fetcher_base.h"
+
+namespace chromeos {
+
+// The AboutAboutSystemLogsFetcher aggregates the unscrubbed logs for display
+// in the about:system page.
+class AboutSystemLogsFetcher
+    : public SystemLogsFetcherBase,
+      public base::SupportsWeakPtr<AboutSystemLogsFetcher> {
+ public:
+  AboutSystemLogsFetcher();
+  ~AboutSystemLogsFetcher();
+
+ private:
+  // Returns a weak ptr to SystemLogsFetcherBase.

[stevenjb, 2013/08/09 18:52:44]

nit: All I was asking for was to document which class this was overriding, e.g:
// SystemLogsFetcherBase

And, now that I think about it, do we really need to implement this as a
virtual, or can it just be implemented in the base class, since it just returns
a WeakPtr to the base class?

[rkc, 2013/08/09 20:17:36]

Hmm, I rethought why we need this and realized that I removed the code I had put
in that needed this class. Originally I had this method because the code I had
added had the derived classes using a WeakPtr and the base class needed a
WeakPtr too; which isn't the case anymore, only the base class needs the
WeakPtr.

Changed it around so the base class itself derives from SupportsWeakPtr and uses
AsWeakPtr instead of this method.

+  virtual base::WeakPtr<SystemLogsFetcherBase> GetWeakPtr() OVERRIDE;
+
+  DISALLOW_COPY_AND_ASSIGN(AboutSystemLogsFetcher);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_SYSTEM_LOGS_ABOUT_SYSTEM_LOGS_FETCHER_H_
+