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

Side by Side Diff: chrome/browser/status_icons/status_icon.cc

Issue 8351004: Add an extra argument to the DisplayBalloon method to support custom notification icons. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixing argument order as suggested. Created 9 years, 2 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/status_icons/status_icon.h" 5 #include "chrome/browser/status_icons/status_icon.h"
6 6
7 #include "ui/base/models/menu_model.h" 7 #include "ui/base/models/menu_model.h"
8 8
9 StatusIcon::StatusIcon() 9 StatusIcon::StatusIcon() {
10 {
11 } 10 }
12 11
13 StatusIcon::~StatusIcon() { 12 StatusIcon::~StatusIcon() {
14 } 13 }
15 14
16 void StatusIcon::AddObserver(Observer* observer) { 15 void StatusIcon::AddObserver(Observer* observer) {
17 observers_.AddObserver(observer); 16 observers_.AddObserver(observer);
18 } 17 }
19 18
20 void StatusIcon::RemoveObserver(Observer* observer) { 19 void StatusIcon::RemoveObserver(Observer* observer) {
21 observers_.RemoveObserver(observer); 20 observers_.RemoveObserver(observer);
22 } 21 }
23 22
24 bool StatusIcon::HasObservers() { 23 bool StatusIcon::HasObservers() {
25 return observers_.size() > 0; 24 return observers_.size() > 0;
26 } 25 }
27 26
28 void StatusIcon::DispatchClickEvent() { 27 void StatusIcon::DispatchClickEvent() {
29 FOR_EACH_OBSERVER(Observer, observers_, OnClicked()); 28 FOR_EACH_OBSERVER(Observer, observers_, OnClicked());
30 } 29 }
31 30
32 void StatusIcon::SetContextMenu(ui::MenuModel* menu) { 31 void StatusIcon::SetContextMenu(ui::MenuModel* menu) {
33 context_menu_contents_.reset(menu); 32 context_menu_contents_.reset(menu);
34 UpdatePlatformContextMenu(menu); 33 UpdatePlatformContextMenu(menu);
35 } 34 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698