Add overflow indicator count on Windows.

chrome/browser/ui/panels/panel_overflow_indicator_view.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_UI_PANELS_PANEL_OVERFLOW_INDICATOR_VIEW_H_
+#define CHROME_BROWSER_UI_PANELS_PANEL_OVERFLOW_INDICATOR_VIEW_H_
+#pragma once
+
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/ui/panels/panel_overflow_indicator.h"
+#include "ui/views/controls/label.h"
+
+namespace gfx {
+class Font;
+}
+namespace views {
+class Widget;
+}
+
+class PanelOverflowIndicatorView : public PanelOverflowIndicator {
+ public:
+  PanelOverflowIndicatorView();
+  virtual ~PanelOverflowIndicatorView();
+
+  // Overridden from OverflowIndicator:
+  virtual void SetCount(int count) OVERRIDE;
+  virtual void SetBounds(const gfx::Rect& bounds) OVERRIDE;
+  virtual int GetHeight() const OVERRIDE;
+  virtual void DrawAttention() OVERRIDE;
+  virtual void StopDrawingAttention() OVERRIDE;
+  virtual int GetCount() const OVERRIDE;
+  virtual bool IsDrawingAttention() const OVERRIDE;
+
+ private:
+  void PaintBackground();
+  void UpdateBounds();
+  void UpdateTitle();
+  void GetFrameMask(const gfx::Rect& rect, gfx::Path* path) const;
+
+  int count_;

[jennb, 2011/12/19 22:24:50]

count_ and is_drawing_attention_ should be the same across all platforms. Should
the base class have these in the basic implementation and call a virtual method
for the subclass to add platform-specific logic?

[jianli, 2011/12/20 01:20:41]

Per discussion, we leave as it is now. Will deal with this if we need to share
the common code for other platforms.

+  bool is_drawing_attention_;
+  scoped_ptr<views::Widget> widget_;
+  views::Label title_;
+
+  DISALLOW_COPY_AND_ASSIGN(PanelOverflowIndicatorView);
+};
+
+#endif  // CHROME_BROWSER_UI_PANELS_PANEL_OVERFLOW_INDICATOR_VIEW_H_