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

Unified Diff: chrome/browser/media/webrtc_browsertest_audio.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
« no previous file with comments | « chrome/browser/media/webrtc_browsertest_audio.h ('k') | chrome/browser/media/webrtc_browsertest_base.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/media/webrtc_browsertest_audio.cc
diff --git a/chrome/browser/media/webrtc_browsertest_audio.cc b/chrome/browser/media/webrtc_browsertest_audio.cc
deleted file mode 100644
index 8d2b58de09c2e12c561f0553d384ddc0d08682e7..0000000000000000000000000000000000000000
--- a/chrome/browser/media/webrtc_browsertest_audio.cc
+++ /dev/null
@@ -1,79 +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/media/webrtc_browsertest_audio.h"
-
-#include <stddef.h>
-
-#include "base/files/file.h"
-#include "base/files/file_path.h"
-#include "media/audio/audio_power_monitor.h"
-#include "media/audio/sounds/wav_audio_handler.h"
-#include "media/base/audio_bus.h"
-#include "media/base/audio_parameters.h"
-
-namespace {
-// Opens |wav_filename|, reads it and loads it as a wav file. This function will
-// bluntly trigger CHECKs if we can't read the file or if it's malformed. The
-// caller takes ownership of the returned data. The size of the data is stored
-// in |read_length|.
-std::unique_ptr<char[]> ReadWavFile(const base::FilePath& wav_filename,
- size_t* file_length) {
- base::File wav_file(
- wav_filename, base::File::FLAG_OPEN | base::File::FLAG_READ);
- if (!wav_file.IsValid()) {
- CHECK(false) << "Failed to read " << wav_filename.value();
- return nullptr;
- }
-
- size_t wav_file_length = wav_file.GetLength();
-
- std::unique_ptr<char[]> data(new char[wav_file_length]);
- size_t read_bytes = wav_file.Read(0, data.get(), wav_file_length);
- if (read_bytes != wav_file_length) {
- LOG(ERROR) << "Failed to read all bytes of " << wav_filename.value();
- return nullptr;
- }
- *file_length = wav_file_length;
- return data;
-}
-} // namespace
-
-namespace test {
-
-float ComputeAudioEnergyForWavFile(const base::FilePath& wav_filename,
- media::AudioParameters* file_parameters) {
- // Read the file, and put its data in a scoped_ptr so it gets deleted later.
- size_t file_length = 0;
- std::unique_ptr<char[]> wav_file_data =
- ReadWavFile(wav_filename, &file_length);
- auto wav_audio_handler = media::WavAudioHandler::Create(
- base::StringPiece(wav_file_data.get(), file_length));
-
- std::unique_ptr<media::AudioBus> audio_bus = media::AudioBus::Create(
- wav_audio_handler->num_channels(), wav_audio_handler->total_frames());
- base::TimeDelta file_duration = wav_audio_handler->GetDuration();
-
- size_t bytes_written;
- wav_audio_handler->CopyTo(audio_bus.get(), 0, &bytes_written);
- CHECK_EQ(bytes_written, wav_audio_handler->data().size())
- << "Expected to write entire file into bus.";
-
- // Set the filter coefficient to the whole file's duration; this will make the
- // power monitor take the entire file into account.
- media::AudioPowerMonitor power_monitor(wav_audio_handler->sample_rate(),
- file_duration);
- power_monitor.Scan(*audio_bus, audio_bus->frames());
-
- file_parameters->Reset(
- media::AudioParameters::AUDIO_PCM_LOW_LATENCY,
- media::GuessChannelLayout(wav_audio_handler->num_channels()),
- wav_audio_handler->sample_rate(), wav_audio_handler->bits_per_sample(),
- wav_audio_handler->total_frames());
- file_parameters->set_channels_for_discrete(wav_audio_handler->num_channels());
-
- return power_monitor.ReadCurrentPowerAndClip().first;
-}
-
-} // namespace test
« no previous file with comments | « chrome/browser/media/webrtc_browsertest_audio.h ('k') | chrome/browser/media/webrtc_browsertest_base.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698