C++ readability review

components/timers/rtc_alarm.cc

Index: components/timers/rtc_alarm.cc
diff --git a/components/timers/rtc_alarm.cc b/components/timers/rtc_alarm.cc
index 9f2750ebfe77a4d8586c0d6807e001de5e14b751..af806ed5681b44cfbb918ac583b5a56f4f2bab39 100644
--- a/components/timers/rtc_alarm.cc
+++ b/components/timers/rtc_alarm.cc
@@ -179,4 +179,5 @@ void RtcAlarm::OnTimerFired(int event_id) {
     parent_->OnTimerFired();

[gromer, 2014/12/05 20:13:41]

Isn't there a race condition where parent_ could be destroyed between these two
lines?

[Chirantan Ekbote, 2014/12/31 00:58:29]

No.  WeakPtrs can only be dereferenced and invalidated on the
SequencedTaskRunner on which they were created.  A SequencedTaskRunner is
essentially a FIFO of tasks.  So parent_ could be invalidated before
OnTimerFired is called and it could be invalidated after OnTimerFired returns
but it could not be invalidated in the middle of OnTimerFired.

 }
 
+
 }  // namespace timers