Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(388)

Unified Diff: trunk/src/chrome/browser/feedback/feedback_report.cc

Issue 149993002: Revert 247772 "Cache feedback reports to disk in case of send fa..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: trunk/src/chrome/browser/feedback/feedback_report.cc
===================================================================
--- trunk/src/chrome/browser/feedback/feedback_report.cc (revision 247814)
+++ trunk/src/chrome/browser/feedback/feedback_report.cc (working copy)
@@ -1,91 +0,0 @@
-// 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_report.h"
-
-#include "base/file_util.h"
-#include "base/files/file_enumerator.h"
-#include "base/files/important_file_writer.h"
-#include "base/guid.h"
-#include "base/strings/string_number_conversions.h"
-#include "content/public/browser/browser_context.h"
-#include "content/public/browser/browser_thread.h"
-#include "net/base/directory_lister.h"
-
-using content::BrowserThread;
-
-namespace {
-
-const base::FilePath::CharType kFeedbackReportPath[] =
- FILE_PATH_LITERAL("Feedback Reports");
-const base::FilePath::CharType kFeedbackReportFilenameWildcard[] =
- FILE_PATH_LITERAL("Feedback Report.*");
-
-const char kFeedbackReportFilenamePrefix[] = "Feedback Report.";
-
-base::FilePath GetFeedbackReportsPath(content::BrowserContext* context) {
- return context->GetPath().Append(kFeedbackReportPath);
-}
-
-} // namespace
-
-namespace feedback {
-
-FeedbackReport::FeedbackReport(content::BrowserContext* context,
- const base::Time& upload_at,
- const std::string& data)
- : context_(context),
- upload_at_(upload_at),
- data_(data) {
- base::FilePath reports_path = GetFeedbackReportsPath(context);
- if (reports_path.empty())
- return;
- file_ = reports_path.AppendASCII(
- kFeedbackReportFilenamePrefix + base::GenerateGUID());
- BrowserThread::PostBlockingPoolTask(FROM_HERE, base::Bind(
- &FeedbackReport::WriteReportOnBlockingPool, this));
-}
-
-FeedbackReport::~FeedbackReport() {}
-
-void FeedbackReport::DeleteReportOnDisk() {
- BrowserThread::PostBlockingPoolTask(FROM_HERE, base::Bind(
- base::IgnoreResult(&base::DeleteFile), file_, false));
- file_.clear();
-}
-
-void FeedbackReport::WriteReportOnBlockingPool() {
- base::FilePath reports_path = GetFeedbackReportsPath(context_);
- if (!base::DirectoryExists(reports_path)) {
- base::File::Error error;
- if (!base::CreateDirectoryAndGetError(reports_path, &error))
- return;
- }
- if (!file_.empty())
- base::ImportantFileWriter::WriteFileAtomically(file_, data_);
-}
-
-// static
-void FeedbackReport::LoadReportsAndQueue(
- content::BrowserContext* context, QueueCallback callback) {
- base::FilePath reports_path = GetFeedbackReportsPath(context);
- if (reports_path.empty())
- return;
-
- base::FileEnumerator enumerator(reports_path,
- false,
- base::FileEnumerator::FILES,
- kFeedbackReportFilenameWildcard);
- for (base::FilePath name = enumerator.Next();
- !name.empty();
- name = enumerator.Next()) {
- std::string data;
- if (ReadFileToString(name, &data))
- BrowserThread::PostTask(
- BrowserThread::UI, FROM_HERE, base::Bind(callback, data));
- base::DeleteFile(name, false);
- }
-}
-
-} // namespace feedback
« no previous file with comments | « trunk/src/chrome/browser/feedback/feedback_report.h ('k') | trunk/src/chrome/browser/feedback/feedback_uploader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698