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

Unified Diff: third_party/WebKit/Source/platform/speech/PlatformSpeechSynthesisUtterance.h

Issue 1617383003: Move PlatformSpeechSynthesisVoice off the heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix oilpan compilation Created 4 years, 11 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
Index: third_party/WebKit/Source/platform/speech/PlatformSpeechSynthesisUtterance.h
diff --git a/third_party/WebKit/Source/platform/speech/PlatformSpeechSynthesisUtterance.h b/third_party/WebKit/Source/platform/speech/PlatformSpeechSynthesisUtterance.h
index f59321dadf51f51a9f07f0005ef6429cacba6626..f86db897082f2c69b7c17d8f50a4e865f6bd53dc 100644
--- a/third_party/WebKit/Source/platform/speech/PlatformSpeechSynthesisUtterance.h
+++ b/third_party/WebKit/Source/platform/speech/PlatformSpeechSynthesisUtterance.h
@@ -51,7 +51,7 @@ public:
const String& lang() const { return m_lang; }
void setLang(const String& lang) { m_lang = lang; }
- PlatformSpeechSynthesisVoice* voice() const { return m_voice; }
+ PlatformSpeechSynthesisVoice* voice() const { return m_voice.get(); }
void setVoice(PlatformSpeechSynthesisVoice* voice) { m_voice = voice; }
// Range = [0, 1] where 1 is the default.
@@ -79,7 +79,7 @@ private:
Member<PlatformSpeechSynthesisUtteranceClient> m_client;
String m_text;
String m_lang;
- Member<PlatformSpeechSynthesisVoice> m_voice;
+ RefPtr<PlatformSpeechSynthesisVoice> m_voice;
float m_volume;
float m_rate;
float m_pitch;

Powered by Google App Engine
This is Rietveld 408576698