Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(15)

Unified Diff: chrome/browser/ui/views/status_icons/status_icon_win.cc

Issue 10869037: Implement status tray notifications in Windows 8 metro chrome using metro style notifications. We d… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/status_icons/status_icon_win.cc
===================================================================
--- chrome/browser/ui/views/status_icons/status_icon_win.cc (revision 152792)
+++ chrome/browser/ui/views/status_icons/status_icon_win.cc (working copy)
@@ -4,6 +4,8 @@
#include "chrome/browser/ui/views/status_icons/status_icon_win.h"
+#include "base/string_number_conversions.h"
+#include "base/win/metro.h"
#include "base/win/windows_version.h"
#include "third_party/skia/include/core/SkBitmap.h"
#include "ui/gfx/icon_util.h"
@@ -158,3 +160,51 @@
icon_data->hWnd = window_;
icon_data->uID = icon_id_;
}
+
+////////////////////////////////////////////////////////////////////////////////
+// StatusIconMetro
+
+StatusIconMetro::StatusIconMetro(UINT id)
+ : id_(id) {
+ DCHECK(base::win::IsMetroProcess());
+}
+
+StatusIconMetro::~StatusIconMetro() {
+}
+
+void StatusIconMetro::SetImage(const SkBitmap& image) {
+ DVLOG(1) << __FUNCTION__;
sky 2012/08/24 16:58:47 Why the DVLOGs?
ananta 2012/08/24 18:03:41 To help debugging cases where this class is used i
+}
+
+void StatusIconMetro::SetPressedImage(const SkBitmap& image) {
+ DVLOG(1) << __FUNCTION__;
+}
+
+void StatusIconMetro::SetToolTip(const string16& tool_tip) {
+ DVLOG(1) << __FUNCTION__;
+ tool_tip_ = tool_tip;
+}
+
+void StatusIconMetro::DisplayBalloon(const SkBitmap& icon,
+ const string16& title,
+ const string16& contents) {
+ DVLOG(1) << __FUNCTION__;
+
+ HMODULE metro_module = base::win::GetMetroModule();
+ DCHECK(metro_module);
+
+ if (metro_module) {
+ base::win::MetroNotification notification =
+ reinterpret_cast<base::win::MetroNotification>(
+ ::GetProcAddress(metro_module, "DisplayNotification"));
+ DCHECK(notification);
+ notification("", "", title.c_str(), contents.c_str(), L"",
+ base::IntToString(id_).c_str());
+ }
+}
+
+void StatusIconMetro::UpdatePlatformContextMenu(ui::MenuModel* menu) {
+ DVLOG(1) << __FUNCTION__
+ << " This functionality is not supported in Windows 8 metro";
+}
+

Powered by Google App Engine
This is Rietveld 408576698