Shows a launch spinner next to a sink once it is selected to create a route. The launch spinner sur…

chrome/browser/ui/webui/media_router/media_router_ui.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_MEDIA_ROUTER_MEDIA_ROUTER_UI_H_
 #define CHROME_BROWSER_UI_WEBUI_MEDIA_ROUTER_MEDIA_ROUTER_UI_H_
 
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 108 matching lines @@
   // Returns the hostname of the default source's parent frame URL.
   std::string GetFrameURLHost() const;
   bool has_pending_route_request() const { return has_pending_route_request_; }
   const GURL& frame_url() const { return frame_url_; }
   const std::vector<MediaSinkWithCastModes>& sinks() const { return sinks_; }
   const std::vector<MediaRoute>& routes() const { return routes_; }
   const std::set<MediaCastMode>& cast_modes() const { return cast_modes_; }
   const content::WebContents* initiator() const { return initiator_; }
 
   const std::string& GetRouteProviderExtensionId() const;
+  const std::set<MediaSink::Id>& GetSinkIdsInLaunch() const;
 
  private:
   class UIIssuesObserver;
   class UIMediaRoutesObserver;
 
   // QueryResultManager::Observer
   void OnResultsUpdated(
       const std::vector<MediaSinkWithCastModes>& sinks) override;
 
   // Called by |issues_observer_| when the top issue has changed.
   // If the UI is already initialized, notifies |handler_| to update the UI.
   // Ignored if the UI is not yet initialized.
   void SetIssue(const Issue* issue);
 
   // Called by |routes_observer_| when the set of active routes has changed.
   void OnRoutesUpdated(const std::vector<MediaRoute>& routes);
 
   // Callback passed to MediaRouter to receive response to route creation
   // requests.
-  void OnRouteResponseReceived(const MediaRoute* route,
+  void OnRouteResponseReceived(const MediaSink::Id& sink_id,
+                               const MediaRoute* route,
                                const std::string& error);
 
   bool DoCreateRoute(const MediaSink::Id& sink_id, MediaCastMode cast_mode);
 
   // Sets the source host name to be displayed in the UI.
   // Gets cast modes from |query_result_manager_| and forwards it to UI.
   // One of the Init* functions must have been called before.
   void UpdateSourceHostAndCastModes(const GURL& frame_url);
 
   // Initializes the dialog with mirroring sources derived from |initiator|,
@@ skipping 16 matching lines @@
   scoped_ptr<MediaRoutesObserver> routes_observer_;
 
   // Set to true by |handler_| when the UI has been initialized.
   bool ui_initialized_;
 
   // Set to |true| if there is a pending route request for this UI.
   bool has_pending_route_request_;
 
   bool requesting_route_for_default_source_;
 
   std::vector<MediaSinkWithCastModes> sinks_;

[Kevin M, 2015/08/08 00:35:56]

Can you put the "is launching" state here?

Or as a bool in the "MediaSinkWithCastModes" struct?

[haibinlu, 2015/08/08 01:25:38]

sinks_ was overwritten when extension updates the list. Unless we let extension
keep tracking the launch states and set the isLaunching boolean in each sink
object, we still need to maintain a launch set here.

Letting extension maintain launch status is fine too. 

   std::vector<MediaRoute> routes_;
   CastModeSet cast_modes_;
   GURL frame_url_;
 
   scoped_ptr<QueryResultManager> query_result_manager_;
 
   // If set, then the result of the next presentation route request will
   // be handled by this object.
   scoped_ptr<CreatePresentationSessionRequest> presentation_request_;
 
@@ skipping 14 matching lines @@
   // NOTE: Weak pointers must be invalidated before all other member variables.
   // Therefore |weak_factory_| must be placed at the end.
   base::WeakPtrFactory<MediaRouterUI> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaRouterUI);
 };
 
 }  // namespace media_router
 
 #endif  // CHROME_BROWSER_UI_WEBUI_MEDIA_ROUTER_MEDIA_ROUTER_UI_H_