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

Side by Side Diff: ui/views/focus/widget_focus_manager.cc

Issue 2418983002: ui/views: convert usage of FOR_EACH_OBSERVER macro (Closed)
Patch Set: Created 4 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
« no previous file with comments | « ui/views/focus/focus_manager.cc ('k') | ui/views/mus/pointer_watcher_event_router.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "ui/views/focus/widget_focus_manager.h" 5 #include "ui/views/focus/widget_focus_manager.h"
6 6
7 #include "base/memory/singleton.h" 7 #include "base/memory/singleton.h"
8 8
9 namespace views { 9 namespace views {
10 10
11 // WidgetFocusManager ---------------------------------------------------------- 11 // WidgetFocusManager ----------------------------------------------------------
12 12
13 // static 13 // static
14 WidgetFocusManager* WidgetFocusManager::GetInstance() { 14 WidgetFocusManager* WidgetFocusManager::GetInstance() {
15 return base::Singleton<WidgetFocusManager>::get(); 15 return base::Singleton<WidgetFocusManager>::get();
16 } 16 }
17 17
18 void WidgetFocusManager::AddFocusChangeListener( 18 void WidgetFocusManager::AddFocusChangeListener(
19 WidgetFocusChangeListener* listener) { 19 WidgetFocusChangeListener* listener) {
20 focus_change_listeners_.AddObserver(listener); 20 focus_change_listeners_.AddObserver(listener);
21 } 21 }
22 22
23 void WidgetFocusManager::RemoveFocusChangeListener( 23 void WidgetFocusManager::RemoveFocusChangeListener(
24 WidgetFocusChangeListener* listener) { 24 WidgetFocusChangeListener* listener) {
25 focus_change_listeners_.RemoveObserver(listener); 25 focus_change_listeners_.RemoveObserver(listener);
26 } 26 }
27 27
28 void WidgetFocusManager::OnNativeFocusChanged(gfx::NativeView focused_now) { 28 void WidgetFocusManager::OnNativeFocusChanged(gfx::NativeView focused_now) {
29 if (enabled_) { 29 if (enabled_) {
30 FOR_EACH_OBSERVER(WidgetFocusChangeListener, focus_change_listeners_, 30 for (WidgetFocusChangeListener& observer : focus_change_listeners_)
31 OnNativeFocusChanged(focused_now)); 31 observer.OnNativeFocusChanged(focused_now);
32 } 32 }
33 } 33 }
34 34
35 WidgetFocusManager::WidgetFocusManager() : enabled_(true) {} 35 WidgetFocusManager::WidgetFocusManager() : enabled_(true) {}
36 36
37 WidgetFocusManager::~WidgetFocusManager() {} 37 WidgetFocusManager::~WidgetFocusManager() {}
38 38
39 // AutoNativeNotificationDisabler ---------------------------------------------- 39 // AutoNativeNotificationDisabler ----------------------------------------------
40 40
41 AutoNativeNotificationDisabler::AutoNativeNotificationDisabler() { 41 AutoNativeNotificationDisabler::AutoNativeNotificationDisabler() {
42 WidgetFocusManager::GetInstance()->DisableNotifications(); 42 WidgetFocusManager::GetInstance()->DisableNotifications();
43 } 43 }
44 44
45 AutoNativeNotificationDisabler::~AutoNativeNotificationDisabler() { 45 AutoNativeNotificationDisabler::~AutoNativeNotificationDisabler() {
46 WidgetFocusManager::GetInstance()->EnableNotifications(); 46 WidgetFocusManager::GetInstance()->EnableNotifications();
47 } 47 }
48 48
49 } // namespace views 49 } // namespace views
OLDNEW
« no previous file with comments | « ui/views/focus/focus_manager.cc ('k') | ui/views/mus/pointer_watcher_event_router.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698