remoting/host/audio_capturer_win.cc - Issue 10823420: Added more error checking for audio packets.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: remoting/host/audio_capturer_win.cc

Issue 10823420: Added more error checking for audio packets. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Addressed comments Created 8 years, 4 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:

Index: remoting/host/audio_capturer_win.cc

diff --git a/remoting/host/audio_capturer_win.cc b/remoting/host/audio_capturer_win.cc

index 03bd426b7747f0af2ad9f0f85d23fc416cca8be5..4dfc174da4aedc96ecf3e875ce526d79c75763bf 100644

--- a/remoting/host/audio_capturer_win.cc

+++ b/remoting/host/audio_capturer_win.cc

@@ -294,10 +294,10 @@ void AudioCapturerWin::DoCapture() {

scoped_ptr<AudioPacket> packet =

scoped_ptr<AudioPacket>(new AudioPacket());

packet->add_data(data, frames * wave_format_ex_->nBlockAlign);

- packet->set_sampling_rate(sampling_rate_);

- packet->set_bytes_per_sample(

- static_cast<AudioPacket::BytesPerSample>(sizeof(int16)));

packet->set_encoding(AudioPacket::ENCODING_RAW);

+ packet->set_sampling_rate(sampling_rate_);

+ packet->set_bytes_per_sample(AudioPacket::BYTES_PER_SAMPLE_2);

+ packet->set_channels(AudioPacket::CHANNELS_STEREO);

callback_.Run(packet.Pass());

}