Address post-commit comments from codereview.chromium.org/371059 and codereview.chromium.org/293019

chrome/browser/cocoa/download_shelf_controller.mm

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/cocoa/download_shelf_controller.h"
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/mac_util.h"
 #include "base/sys_string_conversions.h"
@@ skipping 80 matching lines @@
 }
 
 - (void)dealloc {
   [[NSNotificationCenter defaultCenter] removeObserver:self];
 
   // The controllers will unregister themselves as observers when they are
   // deallocated. No need to do that here.
   [super dealloc];
 }
 
+// Called after the current theme has changed.
 - (void)themeDidChangeNotification:(NSNotification*)aNotification {
   GTMTheme* theme = [aNotification object];
   [self updateTheme:theme];
 }
 
+// Adapt appearance to the current theme. Called after theme changes and before
+// this is shown for the first time.
 - (void)updateTheme:(GTMTheme*)theme {
   // For the default theme, use a blue color for the link. Ideally, we'd want to
   // compare the current theme id with kDefaultThemeID, but the classic theme
   // from the gallery does have a different id. Hence, we use the blue color if
   // the current theme does not change the bookmark text color.
   BOOL useDefaultColor = YES;
   if (bridge_.get() && bridge_->browser() && bridge_->browser()->profile()) {
     ThemeProvider* provider = bridge_->browser()->profile()->GetThemeProvider();
     if (provider)
       useDefaultColor = provider->GetColor(
@@ skipping 178 matching lines @@
     if (isTransferDone &&
         [itemController download]->safety_state() != DownloadItem::DANGEROUS) {
       [self remove:itemController];
     } else {
       ++i;
     }
   }
 }
 
 @end