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

Side by Side Diff: media/audio/audio_parameters.cc

Issue 8786013: Replace media::Limits struct with media::limits namespace and update documentation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix logic Created 9 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/audio/audio_parameters.h" 5 #include "media/audio/audio_parameters.h"
6 6
7 #include "media/base/limits.h" 7 #include "media/base/limits.h"
8 8
9 AudioParameters::AudioParameters() 9 AudioParameters::AudioParameters()
10 : format(AUDIO_PCM_LINEAR), 10 : format(AUDIO_PCM_LINEAR),
(...skipping 10 matching lines...) Expand all
21 : format(format), 21 : format(format),
22 channel_layout(channel_layout), 22 channel_layout(channel_layout),
23 sample_rate(sample_rate), 23 sample_rate(sample_rate),
24 bits_per_sample(bits_per_sample), 24 bits_per_sample(bits_per_sample),
25 samples_per_packet(samples_per_packet), 25 samples_per_packet(samples_per_packet),
26 channels(ChannelLayoutToChannelCount(channel_layout)) { 26 channels(ChannelLayoutToChannelCount(channel_layout)) {
27 } 27 }
28 28
29 bool AudioParameters::IsValid() const { 29 bool AudioParameters::IsValid() const {
30 return (format >= 0) && (format < AUDIO_LAST_FORMAT) && 30 return (format >= 0) && (format < AUDIO_LAST_FORMAT) &&
31 (channels > 0) && (channels <= media::Limits::kMaxChannels) && 31 (channels > 0) && (channels <= media::limits::kMaxChannels) &&
32 (sample_rate > 0) && (sample_rate <= media::Limits::kMaxSampleRate) && 32 (sample_rate > 0) && (sample_rate <= media::limits::kMaxSampleRate) &&
33 (bits_per_sample > 0) && 33 (bits_per_sample > 0) &&
34 (bits_per_sample <= media::Limits::kMaxBitsPerSample) && 34 (bits_per_sample <= media::limits::kMaxBitsPerSample) &&
35 (samples_per_packet > 0) && 35 (samples_per_packet > 0) &&
36 (samples_per_packet <= media::Limits::kMaxSamplesPerPacket); 36 (samples_per_packet <= media::limits::kMaxSamplesPerPacket);
37 } 37 }
38 38
39 int AudioParameters::GetPacketSize() const { 39 int AudioParameters::GetPacketSize() const {
40 return samples_per_packet * channels * bits_per_sample / 8; 40 return samples_per_packet * channels * bits_per_sample / 8;
41 } 41 }
42 42
43 int AudioParameters::GetBytesPerSecond() const { 43 int AudioParameters::GetBytesPerSecond() const {
44 return sample_rate * channels * bits_per_sample / 8; 44 return sample_rate * channels * bits_per_sample / 8;
45 } 45 }
46 46
(...skipping 11 matching lines...) Expand all
58 if (a.sample_rate < b.sample_rate) 58 if (a.sample_rate < b.sample_rate)
59 return true; 59 return true;
60 if (a.sample_rate > b.sample_rate) 60 if (a.sample_rate > b.sample_rate)
61 return false; 61 return false;
62 if (a.bits_per_sample < b.bits_per_sample) 62 if (a.bits_per_sample < b.bits_per_sample)
63 return true; 63 return true;
64 if (a.bits_per_sample > b.bits_per_sample) 64 if (a.bits_per_sample > b.bits_per_sample)
65 return false; 65 return false;
66 return a.samples_per_packet < b.samples_per_packet; 66 return a.samples_per_packet < b.samples_per_packet;
67 } 67 }
OLDNEW
« no previous file with comments | « content/renderer/media/rtc_video_decoder_unittest.cc ('k') | media/audio/win/audio_output_win_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698