content::BrowserContext* browser_context)
: BrowserMessageFilter(TtsMsgStart),
render_process_id_(render_process_id),
- browser_context_(browser_context) {
+ browser_context_(browser_context),
+ valid_(true) {
CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
TtsController::GetInstance()->AddVoicesChangedDelegate(this);
+
+ // Balanced in OnChannelClosingInUIThread() to keep the ref-count be non-zero
+ // until all pointers to this class are invalidated.
+ AddRef();
}
void TtsMessageFilter::OverrideThreadForMessage(
}
void TtsMessageFilter::OnChannelClosing() {
+ base::AutoLock lock(mutex_);
+ valid_ = false;
BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
base::Bind(&TtsMessageFilter::OnChannelClosingInUIThread, this));
}
+bool TtsMessageFilter::Valid() {
+ base::AutoLock lock(mutex_);
+ return valid_;
+}
+
void TtsMessageFilter::OnDestruct() const {
+ {
+ base::AutoLock lock(mutex_);
+ valid_ = false;
+ }
BrowserThread::DeleteOnUIThread::Destruct(this);
}
+TtsMessageFilter::~TtsMessageFilter() {
+ CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+ Cleanup();
+}
+
void TtsMessageFilter::OnInitializeVoiceList() {
CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
TtsController* tts_controller = TtsController::GetInstance();
params.volume = request.volume;
utterance->set_continuous_parameters(params);
- utterance->set_event_delegate(this->AsWeakPtr());
+ utterance->set_event_delegate(this);
TtsController::GetInstance()->SpeakOrEnqueue(utterance.release());
}
TtsEventType event_type,
int char_index,
const std::string& error_message) {
+ if (!Valid())
+ return;
+
CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
switch (event_type) {
case TTS_EVENT_START:
}
void TtsMessageFilter::OnVoicesChanged() {
+ if (!Valid())
+ return;
+
CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
OnInitializeVoiceList();
}
void TtsMessageFilter::OnChannelClosingInUIThread() {
CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- TtsController::GetInstance()->RemoveVoicesChangedDelegate(this);
+ Cleanup();
+ Release(); // Balanced in TtsMessageFilter().
}
-TtsMessageFilter::~TtsMessageFilter() {
- CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+void TtsMessageFilter::Cleanup() {
TtsController::GetInstance()->RemoveVoicesChangedDelegate(this);
+ TtsController::GetInstance()->RemoveUtteranceEventDelegate(this);
}