Replace base::CallbackList with base::ObserverList in CastConfigDelegate

ash/test/tray_cast_test_api.cc

Index: ash/test/tray_cast_test_api.cc
diff --git a/ash/test/tray_cast_test_api.cc b/ash/test/tray_cast_test_api.cc
index 5d0bcf8aab277930581d87407e18bcec6a98b317..a1a2b61850fd3edfe0b4e2e1c8cc85c34ce6a912 100644
--- a/ash/test/tray_cast_test_api.cc
+++ b/ash/test/tray_cast_test_api.cc
@@ -4,7 +4,10 @@
 
 #include "ash/test/tray_cast_test_api.h"
 
+#include "ash/cast_config_delegate.h"
+#include "ash/shell.h"
 #include "ash/system/tray/system_tray.h"
+#include "ash/system/tray/system_tray_delegate.h"
 #include "ui/views/view.h"
 
 namespace ash {
@@ -46,7 +49,11 @@ void TrayCastTestAPI::OnCastingSessionStartedOrStopped(bool is_casting) {
 }
 
 void TrayCastTestAPI::ReleaseConfigCallbacks() {
-  tray_cast_->device_update_subscription_.reset();
+  tray_cast_->added_observer_ = false;
+  ash::Shell::GetInstance()

[achuithb, 2016/01/13 09:21:25]

Do we not need to do the checks for GetInstance and system_tray_delegate() here?

[jdufault, 2016/01/13 19:43:26]

Done, though I'm not sure we need them. This is test code, so I don't think they
should ever be null.

+      ->system_tray_delegate()
+      ->GetCastConfigDelegate()
+      ->RemoveObserver(tray_cast_);
 }
 
 bool TrayCastTestAPI::IsViewDrawn(TrayCast::ChildViewId id) const {