Implement status tray notifications in Windows 8 metro chrome using metro style notifications. We d…

Implement status tray notifications in Windows 8 metro chrome using metro style notifications. We don't
support running native context menu's in these notifications.

BUG=141159
R=sky
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=153303

[sky, 2012-08-24 16:58:46]

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
File chrome/browser/ui/views/status_icons/status_icon_win.cc (right):

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
chrome/browser/ui/views/status_icons/status_icon_win.cc:176: DVLOG(1) <<
__FUNCTION__;
Why the DVLOGs?

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
File chrome/browser/ui/views/status_icons/status_icon_win.h (right):

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
chrome/browser/ui/views/status_icons/status_icon_win.h:95: virtual void
UpdatePlatformContextMenu(ui::MenuModel* menu) OVERRIDE;
Why is this one private, but not the rest?

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
chrome/browser/ui/views/status_icons/status_icon_win.h:98: UINT id_;
const

[ananta, 2012-08-24 18:03:41]

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
File chrome/browser/ui/views/status_icons/status_icon_win.cc (right):

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
chrome/browser/ui/views/status_icons/status_icon_win.cc:176: DVLOG(1) <<
__FUNCTION__;
On 2012/08/24 16:58:47, sky wrote:
> Why the DVLOGs?

To help debugging cases where this class is used in metro mode and things don't
work as expected. We currently don't support functionality like running menu
loops, etc.

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
File chrome/browser/ui/views/status_icons/status_icon_win.h (right):

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
chrome/browser/ui/views/status_icons/status_icon_win.h:95: virtual void
UpdatePlatformContextMenu(ui::MenuModel* menu) OVERRIDE;
On 2012/08/24 16:58:47, sky wrote:
> Why is this one private, but not the rest?

Changed to protected to mimic the way it is declared in the base class
StatusIcon.

https://chromiumcodereview.appspot.com/10869037/diff/5001/chrome/browser/ui/v...
chrome/browser/ui/views/status_icons/status_icon_win.h:98: UINT id_;
On 2012/08/24 16:58:47, sky wrote:
> const

Done.

[sky, 2012-08-24 19:03:35]

LGTM