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

Side by Side Diff: chromecast/media/service/cast_renderer.cc

Issue 2958393002: Add a simple, safe gain PostProcessor. (Closed)
Patch Set: Move cast_audio.json filepath string to a separate file Created 3 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "chromecast/media/service/cast_renderer.h" 5 #include "chromecast/media/service/cast_renderer.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/single_thread_task_runner.h" 8 #include "base/single_thread_task_runner.h"
9 #include "chromecast/base/task_runner_impl.h" 9 #include "chromecast/base/task_runner_impl.h"
10 #include "chromecast/media/base/audio_device_ids.h" 10 #include "chromecast/media/base/audio_device_ids.h"
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
101 // TODO(erickung): crbug.com/443956. Need to provide right LoadType. 101 // TODO(erickung): crbug.com/443956. Need to provide right LoadType.
102 LoadType load_type = kLoadTypeMediaSource; 102 LoadType load_type = kLoadTypeMediaSource;
103 MediaPipelineDeviceParams::MediaSyncType sync_type = 103 MediaPipelineDeviceParams::MediaSyncType sync_type =
104 (load_type == kLoadTypeMediaStream) 104 (load_type == kLoadTypeMediaStream)
105 ? MediaPipelineDeviceParams::kModeIgnorePts 105 ? MediaPipelineDeviceParams::kModeIgnorePts
106 : MediaPipelineDeviceParams::kModeSyncPts; 106 : MediaPipelineDeviceParams::kModeSyncPts;
107 107
108 AudioContentType content_type; 108 AudioContentType content_type;
109 if (audio_device_id_ == kAlarmAudioDeviceId) { 109 if (audio_device_id_ == kAlarmAudioDeviceId) {
110 content_type = AudioContentType::kAlarm; 110 content_type = AudioContentType::kAlarm;
111 } else if (audio_device_id_ == kTtsAudioDeviceId) { 111 } else if (audio_device_id_ == kTtsAudioDeviceId ||
alokp 2017/06/30 17:15:35 Can we delete kTtsAudioDeviceId now?
bshaya 2017/07/06 21:04:45 We still need different DSP treatment for TTS and
112 audio_device_id_ ==
113 ::media::AudioDeviceDescription::kCommunicationsDeviceId) {
112 content_type = AudioContentType::kCommunication; 114 content_type = AudioContentType::kCommunication;
113 } else { 115 } else {
114 content_type = AudioContentType::kMedia; 116 content_type = AudioContentType::kMedia;
115 } 117 }
116 MediaPipelineDeviceParams params(sync_type, backend_task_runner_.get(), 118 MediaPipelineDeviceParams params(sync_type, backend_task_runner_.get(),
117 content_type, audio_device_id_); 119 content_type, audio_device_id_);
118 120
119 if (audio_device_id_ == kTtsAudioDeviceId || 121 if (audio_device_id_ == kTtsAudioDeviceId ||
120 audio_device_id_ == 122 audio_device_id_ ==
121 ::media::AudioDeviceDescription::kCommunicationsDeviceId) { 123 ::media::AudioDeviceDescription::kCommunicationsDeviceId) {
(...skipping 226 matching lines...) Expand 10 before | Expand all | Expand 10 after
348 } 350 }
349 351
350 void CastRenderer::OnVideoOpacityChange(bool opaque) { 352 void CastRenderer::OnVideoOpacityChange(bool opaque) {
351 DCHECK(task_runner_->BelongsToCurrentThread()); 353 DCHECK(task_runner_->BelongsToCurrentThread());
352 DCHECK(opaque); 354 DCHECK(opaque);
353 client_->OnVideoOpacityChange(opaque); 355 client_->OnVideoOpacityChange(opaque);
354 } 356 }
355 357
356 } // namespace media 358 } // namespace media
357 } // namespace chromecast 359 } // namespace chromecast
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698