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

Side by Side Diff: chrome/browser/speech/tts_controller.h

Issue 1014143003: Change set_continuous_parameters() for removing duplicated behavior. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef CHROME_BROWSER_SPEECH_TTS_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_SPEECH_TTS_CONTROLLER_H_
6 #define CHROME_BROWSER_SPEECH_TTS_CONTROLLER_H_ 6 #define CHROME_BROWSER_SPEECH_TTS_CONTROLLER_H_
7 7
8 #include <queue> 8 #include <queue>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 154 matching lines...) Expand 10 before | Expand all | Expand 10 after
165 void set_lang(const std::string& lang) { 165 void set_lang(const std::string& lang) {
166 lang_ = lang; 166 lang_ = lang;
167 } 167 }
168 const std::string& lang() const { return lang_; } 168 const std::string& lang() const { return lang_; }
169 169
170 void set_gender(TtsGenderType gender) { 170 void set_gender(TtsGenderType gender) {
171 gender_ = gender; 171 gender_ = gender;
172 } 172 }
173 TtsGenderType gender() const { return gender_; } 173 TtsGenderType gender() const { return gender_; }
174 174
175 void set_continuous_parameters(const UtteranceContinuousParameters& params) { 175 void set_continuous_parameters(const double rate,
176 continuous_parameters_ = params; 176 const double pitch,
dmazzoni 2015/03/30 16:04:11 Fix the indentation
deejay 2015/03/30 16:29:45 Done.
177 const double volume) {
178 continuous_parameters_.rate = rate;
179 continuous_parameters_.pitch = pitch;
180 continuous_parameters_.volume = volume;
177 } 181 }
178 const UtteranceContinuousParameters& continuous_parameters() { 182 const UtteranceContinuousParameters& continuous_parameters() {
179 return continuous_parameters_; 183 return continuous_parameters_;
180 } 184 }
181 185
182 void set_can_enqueue(bool can_enqueue) { can_enqueue_ = can_enqueue; } 186 void set_can_enqueue(bool can_enqueue) { can_enqueue_ = can_enqueue; }
183 bool can_enqueue() const { return can_enqueue_; } 187 bool can_enqueue() const { return can_enqueue_; }
184 188
185 void set_required_event_types(const std::set<TtsEventType>& types) { 189 void set_required_event_types(const std::set<TtsEventType>& types) {
186 required_event_types_ = types; 190 required_event_types_ = types;
(...skipping 143 matching lines...) Expand 10 before | Expand all | Expand 10 after
330 334
331 // For unit testing. 335 // For unit testing.
332 virtual void SetPlatformImpl(TtsPlatformImpl* platform_impl) = 0; 336 virtual void SetPlatformImpl(TtsPlatformImpl* platform_impl) = 0;
333 virtual int QueueSize() = 0; 337 virtual int QueueSize() = 0;
334 338
335 protected: 339 protected:
336 virtual ~TtsController() {} 340 virtual ~TtsController() {}
337 }; 341 };
338 342
339 #endif // CHROME_BROWSER_SPEECH_TTS_CONTROLLER_H_ 343 #endif // CHROME_BROWSER_SPEECH_TTS_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698