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

ash/test/tray_cast_test_api.cc

 // Copyright 2015 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/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 {
 
 TrayCastTestAPI::TrayCastTestAPI(TrayCast* tray_cast) : tray_cast_(tray_cast) {}
 
 TrayCastTestAPI::~TrayCastTestAPI() {}
 
 bool TrayCastTestAPI::IsTrayInitialized() const {
   return tray_cast_->default_ != nullptr;
@@ skipping 21 matching lines @@
 
 void TrayCastTestAPI::StopCast() {
   return tray_cast_->StopCastForTest();
 }
 
 void TrayCastTestAPI::OnCastingSessionStartedOrStopped(bool is_casting) {
   tray_cast_->OnCastingSessionStartedOrStopped(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 {
   const views::View* view = tray_cast_->GetDefaultView()->GetViewByID(id);
   return view != nullptr && view->IsDrawn();
 }
 
 }  // namespace ash