Fix up status area clock code.

chrome/browser/chromeos/status/status_area_button.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_STATUS_STATUS_AREA_BUTTON_H_
 #define CHROME_BROWSER_CHROMEOS_STATUS_STATUS_AREA_BUTTON_H_
 #pragma once
 
 #include "base/compiler_specific.h"
 #include "base/string16.h"
@@ skipping 56 matching lines @@
   // views::MenuButton overrides.
   virtual bool Activate() OVERRIDE;
 
   // View overrides.
   virtual gfx::Size GetPreferredSize() OVERRIDE;
   virtual gfx::Insets GetInsets() const OVERRIDE;
   virtual void OnThemeChanged() OVERRIDE;
   virtual void SetVisible(bool visible) OVERRIDE;
   virtual bool HitTest(const gfx::Point& l) const OVERRIDE;
 
-  void set_menu_active(bool active) { menu_active_ = active; }
+  virtual void SetMenuActive(bool active);

[oshima, 2011/12/13 06:06:21]

Are you going to use this virtual in separate CL? Will it make sense to document
this as (I assume) it'll do more than just setting menu_active_?

[stevenjb, 2011/12/14 02:19:21]

I'm overriding it in ClockMenuButton. I'll add a comment and make this protected
since it is only called internally.

+
   bool menu_active() const { return menu_active_; }
 
  protected:
   Delegate* delegate() { return delegate_; }
   const Delegate* delegate() const { return delegate_; }
 
   // Subclasses should override these methods to return the correct dimensions.
   virtual int icon_height();
   virtual int icon_width();
 
@@ skipping 11 matching lines @@
 
  private:
   void UpdateTextStyle();
 
   Delegate* delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(StatusAreaButton);
 };
 
 #endif  // CHROME_BROWSER_CHROMEOS_STATUS_STATUS_AREA_BUTTON_H_