Moves the notification icon out of the status area overflow.

chrome/browser/status_icons/status_icon.h

Index: chrome/browser/status_icons/status_icon.h
diff --git a/chrome/browser/status_icons/status_icon.h b/chrome/browser/status_icons/status_icon.h
index 7cb88992914f88f553d786feb0011c1939cf57ff..c8b19ddca75d02cbe344c48fb7f1d37fe1ca0910 100644
--- a/chrome/browser/status_icons/status_icon.h
+++ b/chrome/browser/status_icons/status_icon.h
@@ -17,6 +17,10 @@ class ImageSkia;
 
 class StatusIconObserver;
 
+#if defined(OS_WIN)
+class StatusIconWin;
+#endif  // defined(OS_WIN)
+
 class StatusIcon {
  public:
   StatusIcon();
@@ -56,8 +60,14 @@ class StatusIcon {
 
   // Dispatches a click event to the observers.
   void DispatchClickEvent();
+
 #if defined(OS_WIN)
+  // Dispatches a balloon click event to the observers.
   void DispatchBalloonClickEvent();
+
+  // Available on Windows so clients can access Windows-specific functionality
+  // like the backing HWND.
+  StatusIconWin* AsStatusIconWin();

[sky, 2014/03/19 22:26:39]

This means the parent class knows about a UI specific implementation. In general
we don't want the base classes knowing about specific implementations. You need
to find another way to go about this.

[dewittj, 2014/03/20 19:31:08]

Perhaps atwilson can suggest a better way.  In earlier iterations of this code
I'd used a static_cast<StatusIconWin*>(status_icon) but it seemed safer to avoid
static casts and just return NULL in scenarios where it's unsafe.  The trivial
change is for this to return a bool, and use that to inform whether a
static_cast is accepatble but I doubt that would actually satisfy your concern.

[sky, 2014/03/20 20:59:38]

If you can't actually have your code take a StatusIconWin, then yes, a
static_cast is what you should do.

 #endif
 
  protected: