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

Unified Diff: chrome/browser/media/webrtc_event_log_handler.cc

Issue 2307083002: Cleanup: move WebRTC related files from chrome/browser/media to chrome/browser/media/webrtc/ (Closed)
Patch Set: Removed file wrongly resuscitated during rebase Created 4 years, 3 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: chrome/browser/media/webrtc_event_log_handler.cc
diff --git a/chrome/browser/media/webrtc_event_log_handler.cc b/chrome/browser/media/webrtc_event_log_handler.cc
deleted file mode 100644
index e5f6d49ca9f9f4ee45784d6f7630c8df24d0b9f1..0000000000000000000000000000000000000000
--- a/chrome/browser/media/webrtc_event_log_handler.cc
+++ /dev/null
@@ -1,148 +0,0 @@
-// Copyright 2016 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/media/webrtc_event_log_handler.h"
-
-#include <string>
-#include <utility>
-
-#include "base/bind.h"
-#include "base/command_line.h"
-#include "base/files/file_util.h"
-#include "base/strings/string_number_conversions.h"
-#include "base/time/time.h"
-#include "chrome/browser/media/webrtc_log_list.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/common/chrome_switches.h"
-#include "content/public/browser/browser_thread.h"
-#include "content/public/browser/render_process_host.h"
-
-using content::BrowserThread;
-
-// Keys used to attach handler to the RenderProcessHost
-const char WebRtcEventLogHandler::kWebRtcEventLogHandlerKey[] =
- "kWebRtcEventLogHandlerKey";
-
-namespace {
-
-// Returns a path name to be used as prefix for RTC event log files.
-base::FilePath GetWebRtcEventLogPrefixPath(const base::FilePath& directory,
- uint64_t rtc_event_log_id) {
- static const char kWebRtcEventLogFilePrefix[] = "WebRtcEventLog.";
- return directory.AppendASCII(kWebRtcEventLogFilePrefix +
- base::Uint64ToString(rtc_event_log_id));
-}
-
-} // namespace
-
-WebRtcEventLogHandler::WebRtcEventLogHandler(Profile* profile)
- : profile_(profile),
- current_rtc_event_log_id_(0) {
- DCHECK(profile_);
- thread_checker_.DetachFromThread();
-}
-
-WebRtcEventLogHandler::~WebRtcEventLogHandler() {}
-
-void WebRtcEventLogHandler::StartWebRtcEventLogging(
- content::RenderProcessHost* host,
- base::TimeDelta delay,
- const RecordingDoneCallback& callback,
- const RecordingErrorCallback& error_callback) {
- DCHECK(thread_checker_.CalledOnValidThread());
-
- BrowserThread::PostTaskAndReplyWithResult(
- BrowserThread::FILE, FROM_HERE,
- base::Bind(&WebRtcEventLogHandler::GetLogDirectoryAndEnsureExists, this),
- base::Bind(&WebRtcEventLogHandler::DoStartWebRtcEventLogging, this, host,
- delay, callback, error_callback));
-}
-
-void WebRtcEventLogHandler::StopWebRtcEventLogging(
- content::RenderProcessHost* host,
- const RecordingDoneCallback& callback,
- const RecordingErrorCallback& error_callback) {
- DCHECK(thread_checker_.CalledOnValidThread());
- const bool is_manual_stop = true;
- BrowserThread::PostTaskAndReplyWithResult(
- BrowserThread::FILE, FROM_HERE,
- base::Bind(&WebRtcEventLogHandler::GetLogDirectoryAndEnsureExists, this),
- base::Bind(&WebRtcEventLogHandler::DoStopWebRtcEventLogging, this, host,
- is_manual_stop, current_rtc_event_log_id_, callback,
- error_callback));
-}
-
-base::FilePath WebRtcEventLogHandler::GetLogDirectoryAndEnsureExists() {
- DCHECK_CURRENTLY_ON(BrowserThread::FILE);
- base::FilePath log_dir_path =
- WebRtcLogList::GetWebRtcLogDirectoryForProfile(profile_->GetPath());
- base::File::Error error;
- if (!base::CreateDirectoryAndGetError(log_dir_path, &error)) {
- DLOG(ERROR) << "Could not create WebRTC log directory, error: " << error;
- return base::FilePath();
- }
- return log_dir_path;
-}
-
-void WebRtcEventLogHandler::DoStartWebRtcEventLogging(
- content::RenderProcessHost* host,
- base::TimeDelta delay,
- const RecordingDoneCallback& callback,
- const RecordingErrorCallback& error_callback,
- const base::FilePath& log_directory) {
- DCHECK(thread_checker_.CalledOnValidThread());
-
- base::FilePath prefix_path =
- GetWebRtcEventLogPrefixPath(log_directory, ++current_rtc_event_log_id_);
- if (!host->StartWebRTCEventLog(prefix_path)) {
- error_callback.Run("RTC event logging already in progress");
- return;
- }
-
- if (delay.is_zero()) {
- const bool is_stopped = false, is_manual_stop = false;
- callback.Run(prefix_path.AsUTF8Unsafe(), is_stopped, is_manual_stop);
- return;
- }
-
- const bool is_manual_stop = false;
- BrowserThread::PostDelayedTask(
- BrowserThread::UI, FROM_HERE,
- base::Bind(&WebRtcEventLogHandler::DoStopWebRtcEventLogging, this, host,
- is_manual_stop, current_rtc_event_log_id_, callback,
- error_callback, log_directory),
- delay);
-}
-
-void WebRtcEventLogHandler::DoStopWebRtcEventLogging(
- content::RenderProcessHost* host,
- bool is_manual_stop,
- uint64_t rtc_event_log_id,
- const RecordingDoneCallback& callback,
- const RecordingErrorCallback& error_callback,
- const base::FilePath& log_directory) {
- DCHECK(thread_checker_.CalledOnValidThread());
- DCHECK_LE(rtc_event_log_id, current_rtc_event_log_id_);
-
- base::FilePath prefix_path =
- GetWebRtcEventLogPrefixPath(log_directory, rtc_event_log_id);
- // Prevent an old posted DoStopWebRtcEventLogging() call to stop a newer dump.
- // This could happen in a sequence like:
- // Start(10); // Start dump 1. Post Stop() to run after 10 seconds.
- // Stop(); // Manually stop dump 1 before 10 seconds;
- // Start(20); // Start dump 2. Posted Stop() for 1 should not stop dump 2.
- if (rtc_event_log_id < current_rtc_event_log_id_) {
- const bool is_stopped = false;
- callback.Run(prefix_path.AsUTF8Unsafe(), is_stopped, is_manual_stop);
- return;
- }
-
- if (!host->StopWebRTCEventLog()) {
- error_callback.Run("No RTC event logging in progress");
- return;
- }
-
- const bool is_stopped = true;
- callback.Run(prefix_path.AsUTF8Unsafe(), is_stopped, is_manual_stop);
-}
« no previous file with comments | « chrome/browser/media/webrtc_event_log_handler.h ('k') | chrome/browser/media/webrtc_getmediadevices_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698