Last Comment Bug 778296 - Add comment to nsITimer.idl about thread-safety
: Add comment to nsITimer.idl about thread-safety
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: General (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla17
Assigned To: Nobody; OK to take it and work on it
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-27 13:58 PDT by Jason Duell [:jduell] (needinfo? me)
Modified: 2012-07-28 18:35 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
v1 (1.25 KB, patch)
2012-07-27 13:58 PDT, Jason Duell [:jduell] (needinfo? me)
bzbarsky: review+
Details | Diff | Review

Description Jason Duell [:jduell] (needinfo? me) 2012-07-27 13:58:26 PDT
Created attachment 646714 [details] [diff] [review]
v1

per dev.platform discussion a while back...

Assume just a +r is needed here, or do we want +sr too?  (seems like overkill for comment-only change)
Comment 1 Boris Zbarsky [:bz] (Out June 25-July 6) 2012-07-27 14:16:35 PDT
Comment on attachment 646714 [details] [diff] [review]
v1

r=me.

I don't think this needs sr.
Comment 2 Jason Duell [:jduell] (needinfo? me) 2012-07-27 15:36:02 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/6d00e5bf736a
Comment 3 Jason Duell [:jduell] (needinfo? me) 2012-07-27 15:42:53 PDT
Somehow managed to check in an empty commit in previous comment

  https://hg.mozilla.org/integration/mozilla-inbound/rev/fa933ff1f137

Note You need to log in before you can comment on or make changes to this bug.