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

Side by Side Diff: chrome/browser/ui/webui/media_router/media_router_ui.cc

Issue 2874283002: [Media Router] Notify MRUI of a media status update when it starts observing (Closed)
Patch Set: Created 3 years, 7 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "chrome/browser/ui/webui/media_router/media_router_ui.h" 5 #include "chrome/browser/ui/webui/media_router/media_router_ui.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <string> 8 #include <string>
9 #include <unordered_map> 9 #include <unordered_map>
10 #include <utility> 10 #include <utility>
(...skipping 139 matching lines...) Expand 10 before | Expand all | Expand 10 after
150 150
151 void MediaRouterUI::UIMediaRoutesObserver::OnRoutesUpdated( 151 void MediaRouterUI::UIMediaRoutesObserver::OnRoutesUpdated(
152 const std::vector<MediaRoute>& routes, 152 const std::vector<MediaRoute>& routes,
153 const std::vector<MediaRoute::Id>& joinable_route_ids) { 153 const std::vector<MediaRoute::Id>& joinable_route_ids) {
154 callback_.Run(routes, joinable_route_ids); 154 callback_.Run(routes, joinable_route_ids);
155 } 155 }
156 156
157 MediaRouterUI::UIMediaRouteControllerObserver::UIMediaRouteControllerObserver( 157 MediaRouterUI::UIMediaRouteControllerObserver::UIMediaRouteControllerObserver(
158 MediaRouterUI* ui, 158 MediaRouterUI* ui,
159 scoped_refptr<MediaRouteController> controller) 159 scoped_refptr<MediaRouteController> controller)
160 : MediaRouteController::Observer(std::move(controller)), ui_(ui) {} 160 : MediaRouteController::Observer(std::move(controller)), ui_(ui) {
161 CheckForMediaStatusUpdates();
mark a. foltz 2017/05/12 21:00:58 Instead you could call OnMediaStatusUpdated(contro
takumif 2017/05/12 22:20:58 That would require current_media_status() to be pu
162 }
163
161 MediaRouterUI::UIMediaRouteControllerObserver:: 164 MediaRouterUI::UIMediaRouteControllerObserver::
162 ~UIMediaRouteControllerObserver() {} 165 ~UIMediaRouteControllerObserver() {}
163 166
164 void MediaRouterUI::UIMediaRouteControllerObserver::OnMediaStatusUpdated( 167 void MediaRouterUI::UIMediaRouteControllerObserver::OnMediaStatusUpdated(
165 const MediaStatus& status) { 168 const MediaStatus& status) {
166 ui_->UpdateMediaRouteStatus(status); 169 ui_->UpdateMediaRouteStatus(status);
167 } 170 }
168 171
169 void MediaRouterUI::UIMediaRouteControllerObserver::OnControllerInvalidated() { 172 void MediaRouterUI::UIMediaRouteControllerObserver::OnControllerInvalidated() {
170 ui_->OnRouteControllerInvalidated(); 173 ui_->OnRouteControllerInvalidated();
(...skipping 640 matching lines...) Expand 10 before | Expand all | Expand 10 after
811 } 814 }
812 815
813 std::string MediaRouterUI::GetSerializedInitiatorOrigin() const { 816 std::string MediaRouterUI::GetSerializedInitiatorOrigin() const {
814 url::Origin origin = initiator_ 817 url::Origin origin = initiator_
815 ? url::Origin(initiator_->GetLastCommittedURL()) 818 ? url::Origin(initiator_->GetLastCommittedURL())
816 : url::Origin(); 819 : url::Origin();
817 return origin.Serialize(); 820 return origin.Serialize();
818 } 821 }
819 822
820 } // namespace media_router 823 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698