Cache feedback reports to disk in case of send failure.

chrome/browser/feedback/feedback_profile_observer.cc

+// Copyright 2014 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/feedback/feedback_profile_observer.h"
+
+#include "base/callback.h"
+#include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/feedback/feedback_report.h"
+#include "chrome/browser/feedback/feedback_uploader.h"
+#include "chrome/browser/feedback/feedback_uploader_factory.h"
+#include "chrome/browser/profiles/profile.h"
+#include "content/public/browser/browser_context.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/notification_service.h"
+
+using content::BrowserThread;
+
+namespace {
+
+static base::LazyInstance<feedback::FeedbackProfileObserver>::Leaky

[Lei Zhang, 2014/01/23 22:28:41]

You don't need static inside an anonymous namespace.

[rkc, 2014/01/23 22:57:21]

Done.

+    g_feedback_profile_observer = LAZY_INSTANCE_INITIALIZER;
+
+}
+
+namespace feedback {
+
+// static
+void FeedbackProfileObserver::Initialize() {
+  g_feedback_profile_observer.Get();
+}
+
+FeedbackProfileObserver::FeedbackProfileObserver() {
+  prefs_registrar_.Add(this, chrome::NOTIFICATION_PROFILE_CREATED,
+                       content::NotificationService::AllSources());
+}
+
+FeedbackProfileObserver::~FeedbackProfileObserver() {
+  prefs_registrar_.RemoveAll();
+}
+
+void FeedbackProfileObserver::Observe(
+    int type,
+    const content::NotificationSource& source,
+    const content::NotificationDetails& details) {
+  switch (type) {

[Lei Zhang, 2014/01/23 22:28:41]

Don't bother with the switch(). Just
DCHECK_EQ(chrome::NOTIFICATION_PROFILE_CREATED, type);

[rkc, 2014/01/23 22:57:21]

Done.

+    case chrome::NOTIFICATION_PROFILE_CREATED: {
+      DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+      Profile* profile = content::Source<Profile>(source).ptr();
+      if (!profile->IsOffTheRecord())
+        QueueUnsentReports(profile);
+      break;
+    }
+    default:
+      NOTREACHED();
+  }
+}
+
+void FeedbackProfileObserver::QueueUnsentReports(
+    content::BrowserContext* context) {
+  feedback::FeedbackUploader *uploader =

[Lei Zhang, 2014/01/23 22:28:41]

nit: feedback::FeedbackUploader*

[rkc, 2014/01/23 22:57:21]

Done.

+      feedback::FeedbackUploaderFactory::GetForBrowserContext(context);
+  BrowserThread::PostBlockingPoolTask(FROM_HERE,
+      base::Bind(
+          &FeedbackReport::LoadReportsAndQueue, context, base::Bind(
+              &FeedbackUploader::QueueReport, uploader->AsWeakPtr())));
+}
+
+}  // namespace feedback