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

Side by Side Diff: chrome/browser/media/router/presentation_service_delegate_impl.cc

Issue 1466573002: Presentation API: implement renderer side of PresentationConnection.close(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years 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/media/router/presentation_service_delegate_impl.h" 5 #include "chrome/browser/media/router/presentation_service_delegate_impl.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/containers/scoped_ptr_hash_map.h" 9 #include "base/containers/scoped_ptr_hash_map.h"
10 #include "base/containers/small_map.h" 10 #include "base/containers/small_map.h"
(...skipping 707 matching lines...) Expand 10 before | Expand all | Expand 10 after
718 void PresentationServiceDelegateImpl::CloseSession( 718 void PresentationServiceDelegateImpl::CloseSession(
719 int render_process_id, 719 int render_process_id,
720 int render_frame_id, 720 int render_frame_id,
721 const std::string& presentation_id) { 721 const std::string& presentation_id) {
722 const MediaRoute::Id& route_id = frame_manager_->GetRouteId( 722 const MediaRoute::Id& route_id = frame_manager_->GetRouteId(
723 RenderFrameHostId(render_process_id, render_frame_id), presentation_id); 723 RenderFrameHostId(render_process_id, render_frame_id), presentation_id);
724 if (route_id.empty()) { 724 if (route_id.empty()) {
725 DVLOG(1) << "No active route for: " << presentation_id; 725 DVLOG(1) << "No active route for: " << presentation_id;
726 return; 726 return;
727 } 727 }
728
729 // TODO(mfoltz, mlamouri): implement CloseSession().
730 // This could call router_->OnPresentationSessionDetached(route_id).
731 // PresentationFrame::Reset() should probably call CloseSession() too.
732 // Rename CloseRoute() to something else to avoid confusion?
733 NOTIMPLEMENTED();
734 }
735
736 void PresentationServiceDelegateImpl::TerminateSession(
737 int render_process_id,
738 int render_frame_id,
739 const std::string& presentation_id) {
740 const MediaRoute::Id& route_id = frame_manager_->GetRouteId(
741 RenderFrameHostId(render_process_id, render_frame_id), presentation_id);
742 if (route_id.empty()) {
743 DVLOG(1) << "No active route for: " << presentation_id;
744 return;
745 }
728 router_->CloseRoute(route_id); 746 router_->CloseRoute(route_id);
729 } 747 }
730 748
731 void PresentationServiceDelegateImpl::ListenForSessionMessages( 749 void PresentationServiceDelegateImpl::ListenForSessionMessages(
732 int render_process_id, 750 int render_process_id,
733 int render_frame_id, 751 int render_frame_id,
734 const content::PresentationSessionInfo& session, 752 const content::PresentationSessionInfo& session,
735 const content::PresentationSessionMessageCallback& message_cb) { 753 const content::PresentationSessionMessageCallback& message_cb) {
736 frame_manager_->ListenForSessionMessages( 754 frame_manager_->ListenForSessionMessages(
737 RenderFrameHostId(render_process_id, render_frame_id), session, 755 RenderFrameHostId(render_process_id, render_frame_id), session,
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
820 bool PresentationServiceDelegateImpl::HasScreenAvailabilityListenerForTest( 838 bool PresentationServiceDelegateImpl::HasScreenAvailabilityListenerForTest(
821 int render_process_id, 839 int render_process_id,
822 int render_frame_id, 840 int render_frame_id,
823 const MediaSource::Id& source_id) const { 841 const MediaSource::Id& source_id) const {
824 RenderFrameHostId render_frame_host_id(render_process_id, render_frame_id); 842 RenderFrameHostId render_frame_host_id(render_process_id, render_frame_id);
825 return frame_manager_->HasScreenAvailabilityListenerForTest( 843 return frame_manager_->HasScreenAvailabilityListenerForTest(
826 render_frame_host_id, source_id); 844 render_frame_host_id, source_id);
827 } 845 }
828 846
829 } // namespace media_router 847 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698