chromeos: Remove OS ifdefs from ash/common/system/tray

ash/common/system/tray/system_tray_notifier.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/common/system/tray/system_tray_notifier.h"
 
 #include "ash/common/system/accessibility_observer.h"
-#include "ash/common/system/date/clock_observer.h"
-#include "ash/common/system/ime/ime_observer.h"
-#include "ash/common/system/user/user_observer.h"
-
-#if defined(OS_CHROMEOS)
 #include "ash/common/system/chromeos/bluetooth/bluetooth_observer.h"
 #include "ash/common/system/chromeos/enterprise/enterprise_domain_observer.h"
 #include "ash/common/system/chromeos/network/network_observer.h"
 #include "ash/common/system/chromeos/network/network_portal_detector_observer.h"
 #include "ash/common/system/chromeos/screen_security/screen_capture_observer.h"
 #include "ash/common/system/chromeos/screen_security/screen_share_observer.h"
 #include "ash/common/system/chromeos/session/last_window_closed_observer.h"
 #include "ash/common/system/chromeos/session/logout_button_observer.h"
 #include "ash/common/system/chromeos/session/session_length_limit_observer.h"
 #include "ash/common/system/chromeos/tray_tracing.h"
 #include "ash/common/system/chromeos/virtual_keyboard/virtual_keyboard_observer.h"
-#endif
+#include "ash/common/system/date/clock_observer.h"
+#include "ash/common/system/ime/ime_observer.h"
+#include "ash/common/system/user/user_observer.h"
 
 namespace ash {
 
 SystemTrayNotifier::SystemTrayNotifier() {}
 
 SystemTrayNotifier::~SystemTrayNotifier() {}
 
 void SystemTrayNotifier::AddAccessibilityObserver(
     AccessibilityObserver* observer) {
   accessibility_observers_.AddObserver(observer);
 }
 
 void SystemTrayNotifier::RemoveAccessibilityObserver(
     AccessibilityObserver* observer) {
   accessibility_observers_.RemoveObserver(observer);
 }
 
 void SystemTrayNotifier::NotifyAccessibilityModeChanged(
     AccessibilityNotificationVisibility notify) {
   for (auto& observer : accessibility_observers_)
     observer.OnAccessibilityModeChanged(notify);
 }
 
+void SystemTrayNotifier::AddBluetoothObserver(BluetoothObserver* observer) {
+  bluetooth_observers_.AddObserver(observer);
+}
+
+void SystemTrayNotifier::RemoveBluetoothObserver(BluetoothObserver* observer) {
+  bluetooth_observers_.RemoveObserver(observer);
+}
+
+void SystemTrayNotifier::NotifyRefreshBluetooth() {
+  for (auto& observer : bluetooth_observers_)
+    observer.OnBluetoothRefresh();
+}
+
+void SystemTrayNotifier::NotifyBluetoothDiscoveringChanged() {
+  for (auto& observer : bluetooth_observers_)
+    observer.OnBluetoothDiscoveringChanged();
+}
+
+void SystemTrayNotifier::AddEnterpriseDomainObserver(

[msw, 2016/12/14 22:51:00]

nit: this belongs below with the other enterprise functions.

[James Cook, 2016/12/14 23:46:08]

Gack. Thanks for catching this.

+    EnterpriseDomainObserver* observer) {
+  enterprise_domain_observers_.AddObserver(observer);
+}
+
 void SystemTrayNotifier::AddClockObserver(ClockObserver* observer) {
   clock_observers_.AddObserver(observer);
 }
 
 void SystemTrayNotifier::RemoveClockObserver(ClockObserver* observer) {
   clock_observers_.RemoveObserver(observer);
 }
 
 void SystemTrayNotifier::NotifyRefreshClock() {
   for (auto& observer : clock_observers_)
     observer.Refresh();
 }
 
 void SystemTrayNotifier::NotifyDateFormatChanged() {
   for (auto& observer : clock_observers_)
     observer.OnDateFormatChanged();
 }
 
 void SystemTrayNotifier::NotifySystemClockTimeUpdated() {
   for (auto& observer : clock_observers_)
     observer.OnSystemClockTimeUpdated();
 }
 
 void SystemTrayNotifier::NotifySystemClockCanSetTimeChanged(bool can_set_time) {
   for (auto& observer : clock_observers_)
     observer.OnSystemClockCanSetTimeChanged(can_set_time);
 }
 
+void SystemTrayNotifier::RemoveEnterpriseDomainObserver(
+    EnterpriseDomainObserver* observer) {
+  enterprise_domain_observers_.RemoveObserver(observer);
+}
+
+void SystemTrayNotifier::NotifyEnterpriseDomainChanged() {
+  for (auto& observer : enterprise_domain_observers_)
+    observer.OnEnterpriseDomainChanged();
+}
+
 void SystemTrayNotifier::AddIMEObserver(IMEObserver* observer) {
   ime_observers_.AddObserver(observer);
 }
 
 void SystemTrayNotifier::RemoveIMEObserver(IMEObserver* observer) {
   ime_observers_.RemoveObserver(observer);
 }
 
 void SystemTrayNotifier::NotifyRefreshIME() {
   for (auto& observer : ime_observers_)
     observer.OnIMERefresh();
 }
 
 void SystemTrayNotifier::NotifyRefreshIMEMenu(bool is_active) {
   for (auto& observer : ime_observers_)
     observer.OnIMEMenuActivationChanged(is_active);
 }
 
-void SystemTrayNotifier::AddUserObserver(UserObserver* observer) {
-  user_observers_.AddObserver(observer);
-}
-
-void SystemTrayNotifier::RemoveUserObserver(UserObserver* observer) {
-  user_observers_.RemoveObserver(observer);
-}
-
-void SystemTrayNotifier::NotifyUserUpdate() {
-  for (auto& observer : user_observers_)
-    observer.OnUserUpdate();
-}
-
-void SystemTrayNotifier::NotifyUserAddedToSession() {
-  for (auto& observer : user_observers_)
-    observer.OnUserAddedToSession();
-}
-
-////////////////////////////////////////////////////////////////////////////////
-
-#if defined(OS_CHROMEOS)
-
-void SystemTrayNotifier::AddBluetoothObserver(BluetoothObserver* observer) {
-  bluetooth_observers_.AddObserver(observer);
-}
-
-void SystemTrayNotifier::RemoveBluetoothObserver(BluetoothObserver* observer) {
-  bluetooth_observers_.RemoveObserver(observer);
-}
-
-void SystemTrayNotifier::NotifyRefreshBluetooth() {
-  for (auto& observer : bluetooth_observers_)
-    observer.OnBluetoothRefresh();
-}
-
-void SystemTrayNotifier::NotifyBluetoothDiscoveringChanged() {
-  for (auto& observer : bluetooth_observers_)
-    observer.OnBluetoothDiscoveringChanged();
-}
-
-void SystemTrayNotifier::AddEnterpriseDomainObserver(
-    EnterpriseDomainObserver* observer) {
-  enterprise_domain_observers_.AddObserver(observer);
-}
-
-void SystemTrayNotifier::RemoveEnterpriseDomainObserver(
-    EnterpriseDomainObserver* observer) {
-  enterprise_domain_observers_.RemoveObserver(observer);
-}
-
-void SystemTrayNotifier::NotifyEnterpriseDomainChanged() {
-  for (auto& observer : enterprise_domain_observers_)
-    observer.OnEnterpriseDomainChanged();
-}
-
 void SystemTrayNotifier::AddLastWindowClosedObserver(
     LastWindowClosedObserver* observer) {
   last_window_closed_observers_.AddObserver(observer);
 }
 
 void SystemTrayNotifier::RemoveLastWindowClosedObserver(
     LastWindowClosedObserver* observer) {
   last_window_closed_observers_.RemoveObserver(observer);
 }
 
@@ skipping 121 matching lines @@
 
 void SystemTrayNotifier::RemoveTracingObserver(TracingObserver* observer) {
   tracing_observers_.RemoveObserver(observer);
 }
 
 void SystemTrayNotifier::NotifyTracingModeChanged(bool value) {
   for (auto& observer : tracing_observers_)
     observer.OnTracingModeChanged(value);
 }
 
+void SystemTrayNotifier::AddUserObserver(UserObserver* observer) {
+  user_observers_.AddObserver(observer);
+}
+
+void SystemTrayNotifier::RemoveUserObserver(UserObserver* observer) {
+  user_observers_.RemoveObserver(observer);
+}
+
+void SystemTrayNotifier::NotifyUserUpdate() {
+  for (auto& observer : user_observers_)
+    observer.OnUserUpdate();
+}
+
+void SystemTrayNotifier::NotifyUserAddedToSession() {
+  for (auto& observer : user_observers_)
+    observer.OnUserAddedToSession();
+}
+
 void SystemTrayNotifier::AddVirtualKeyboardObserver(
     VirtualKeyboardObserver* observer) {
   virtual_keyboard_observers_.AddObserver(observer);
 }
 
 void SystemTrayNotifier::RemoveVirtualKeyboardObserver(
     VirtualKeyboardObserver* observer) {
   virtual_keyboard_observers_.RemoveObserver(observer);
 }
 
 void SystemTrayNotifier::NotifyVirtualKeyboardSuppressionChanged(
     bool suppressed) {
   for (auto& observer : virtual_keyboard_observers_)
     observer.OnKeyboardSuppressionChanged(suppressed);
 }
 
-#endif  // defined(OS_CHROMEOS)
-
 }  // namespace ash