diff --git a/camino/src/download/ProgressDlgController.mm b/camino/src/download/ProgressDlgController.mm
index 9b8fbaff48a..469d7b918bd 100644
--- a/camino/src/download/ProgressDlgController.mm
+++ b/camino/src/download/ProgressDlgController.mm
@@ -512,6 +512,8 @@ static id gSharedProgressController = nil;
 - (void)setupDownloadTimer
 {
   [self killDownloadTimer];
+  // note that this sets up a retain cycle between |self| and the timer,
+  // which has to be broken out of band, before we'll be dealloc'd.
   mDownloadTimer = [[NSTimer scheduledTimerWithTimeInterval:1.0
                                                      target:self
                                                    selector:@selector(setDownloadProgress:)