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

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

Issue 2627463003: Convert MediaRouter mojom apis to intake url::Origin objects instead of strings (Closed)
Patch Set: return an Origin instead of GURL for GetLastCommittedURLForFrame Created 3 years, 11 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/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 #include <unordered_map> 8 #include <unordered_map>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 50
51 // Returns the unique identifier for the supplied RenderFrameHost. 51 // Returns the unique identifier for the supplied RenderFrameHost.
52 RenderFrameHostId GetRenderFrameHostId(RenderFrameHost* render_frame_host) { 52 RenderFrameHostId GetRenderFrameHostId(RenderFrameHost* render_frame_host) {
53 int render_process_id = render_frame_host->GetProcess()->GetID(); 53 int render_process_id = render_frame_host->GetProcess()->GetID();
54 int render_frame_id = render_frame_host->GetRoutingID(); 54 int render_frame_id = render_frame_host->GetRoutingID();
55 return RenderFrameHostId(render_process_id, render_frame_id); 55 return RenderFrameHostId(render_process_id, render_frame_id);
56 } 56 }
57 57
58 // Gets the last committed URL for the render frame specified by 58 // Gets the last committed URL for the render frame specified by
59 // |render_frame_host_id|. 59 // |render_frame_host_id|.
60 GURL GetLastCommittedURLForFrame(RenderFrameHostId render_frame_host_id) { 60 url::Origin GetLastCommittedURLForFrame(
61 RenderFrameHostId render_frame_host_id) {
61 RenderFrameHost* render_frame_host = RenderFrameHost::FromID( 62 RenderFrameHost* render_frame_host = RenderFrameHost::FromID(
62 render_frame_host_id.first, render_frame_host_id.second); 63 render_frame_host_id.first, render_frame_host_id.second);
63 if (!render_frame_host) 64 if (!render_frame_host)
64 return GURL(); 65 return url::Origin();
65 66
66 // TODO(crbug.com/632623): Use url::Origin in place of GURL for origins 67 return render_frame_host->GetLastCommittedOrigin();
67 return render_frame_host->GetLastCommittedOrigin().GetURL();
68 } 68 }
69 69
70 // Observes messages originating from the MediaSink connected to a MediaRoute 70 // Observes messages originating from the MediaSink connected to a MediaRoute
71 // that represents a presentation. Converts the messages into 71 // that represents a presentation. Converts the messages into
72 // content::PresentationSessionMessages and dispatches them via the provided 72 // content::PresentationSessionMessages and dispatches them via the provided
73 // PresentationSessionMessageCallback. 73 // PresentationSessionMessageCallback.
74 class PresentationSessionMessagesObserver : public RouteMessageObserver { 74 class PresentationSessionMessagesObserver : public RouteMessageObserver {
75 public: 75 public:
76 // |message_cb|: The callback to invoke whenever messages are received. 76 // |message_cb|: The callback to invoke whenever messages are received.
77 // |route_id|: ID of MediaRoute to listen for messages. 77 // |route_id|: ID of MediaRoute to listen for messages.
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
229 229
230 bool PresentationFrame::SetScreenAvailabilityListener( 230 bool PresentationFrame::SetScreenAvailabilityListener(
231 content::PresentationScreenAvailabilityListener* listener) { 231 content::PresentationScreenAvailabilityListener* listener) {
232 MediaSource source(GetMediaSourceFromListener(listener)); 232 MediaSource source(GetMediaSourceFromListener(listener));
233 auto& sinks_observer = url_to_sinks_observer_[source.id()]; 233 auto& sinks_observer = url_to_sinks_observer_[source.id()];
234 if (sinks_observer && sinks_observer->listener() == listener) 234 if (sinks_observer && sinks_observer->listener() == listener)
235 return false; 235 return false;
236 236
237 sinks_observer.reset(new PresentationMediaSinksObserver( 237 sinks_observer.reset(new PresentationMediaSinksObserver(
238 router_, listener, source, 238 router_, listener, source,
239 GetLastCommittedURLForFrame(render_frame_host_id_).GetOrigin())); 239 GetLastCommittedURLForFrame(render_frame_host_id_)));
240 240
241 if (!sinks_observer->Init()) { 241 if (!sinks_observer->Init()) {
242 url_to_sinks_observer_.erase(source.id()); 242 url_to_sinks_observer_.erase(source.id());
243 listener->OnScreenAvailabilityNotSupported(); 243 listener->OnScreenAvailabilityNotSupported();
244 return false; 244 return false;
245 } 245 }
246 246
247 return true; 247 return true;
248 } 248 }
249 249
(...skipping 303 matching lines...) Expand 10 before | Expand all | Expand 10 after
553 const RenderFrameHostId& render_frame_host_id, 553 const RenderFrameHostId& render_frame_host_id,
554 const std::vector<GURL>& default_presentation_urls, 554 const std::vector<GURL>& default_presentation_urls,
555 const content::PresentationSessionStartedCallback& callback) { 555 const content::PresentationSessionStartedCallback& callback) {
556 if (!IsMainFrame(render_frame_host_id)) 556 if (!IsMainFrame(render_frame_host_id))
557 return; 557 return;
558 558
559 if (default_presentation_urls.empty()) { 559 if (default_presentation_urls.empty()) {
560 ClearDefaultPresentationRequest(); 560 ClearDefaultPresentationRequest();
561 } else { 561 } else {
562 DCHECK(!callback.is_null()); 562 DCHECK(!callback.is_null());
563 GURL frame_url(GetLastCommittedURLForFrame(render_frame_host_id)); 563 url::Origin frame_origin =
mark a. foltz 2017/01/24 04:04:18 const auto& frame_origin
steimel 2017/01/25 19:01:11 Done.
564 GetLastCommittedURLForFrame(render_frame_host_id);
564 PresentationRequest request(render_frame_host_id, default_presentation_urls, 565 PresentationRequest request(render_frame_host_id, default_presentation_urls,
565 frame_url); 566 frame_origin);
566 default_presentation_started_callback_ = callback; 567 default_presentation_started_callback_ = callback;
567 SetDefaultPresentationRequest(request); 568 SetDefaultPresentationRequest(request);
568 } 569 }
569 } 570 }
570 571
571 void PresentationFrameManager::AddDelegateObserver( 572 void PresentationFrameManager::AddDelegateObserver(
572 const RenderFrameHostId& render_frame_host_id, 573 const RenderFrameHostId& render_frame_host_id,
573 DelegateObserver* observer) { 574 DelegateObserver* observer) {
574 auto* presentation_frame = GetOrAddPresentationFrame(render_frame_host_id); 575 auto* presentation_frame = GetOrAddPresentationFrame(render_frame_host_id);
575 presentation_frame->set_delegate_observer(observer); 576 presentation_frame->set_delegate_observer(observer);
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
824 const content::PresentationSessionStartedCallback& success_cb, 825 const content::PresentationSessionStartedCallback& success_cb,
825 const content::PresentationSessionErrorCallback& error_cb) { 826 const content::PresentationSessionErrorCallback& error_cb) {
826 DVLOG(2) << "PresentationServiceDelegateImpl::JoinSession"; 827 DVLOG(2) << "PresentationServiceDelegateImpl::JoinSession";
827 if (presentation_urls.empty()) { 828 if (presentation_urls.empty()) {
828 error_cb.Run(content::PresentationError( 829 error_cb.Run(content::PresentationError(
829 content::PRESENTATION_ERROR_NO_PRESENTATION_FOUND, 830 content::PRESENTATION_ERROR_NO_PRESENTATION_FOUND,
830 "Invalid presentation arguments.")); 831 "Invalid presentation arguments."));
831 return; 832 return;
832 } 833 }
833 834
834 const url::Origin& origin = url::Origin(GetLastCommittedURLForFrame( 835 const url::Origin& origin = GetLastCommittedURLForFrame(
835 RenderFrameHostId(render_process_id, render_frame_id))); 836 RenderFrameHostId(render_process_id, render_frame_id));
836 837
837 #if !defined(OS_ANDROID) 838 #if !defined(OS_ANDROID)
838 if (IsAutoJoinPresentationId(presentation_id) && 839 if (IsAutoJoinPresentationId(presentation_id) &&
839 ShouldCancelAutoJoinForOrigin(origin)) { 840 ShouldCancelAutoJoinForOrigin(origin)) {
840 error_cb.Run(content::PresentationError( 841 error_cb.Run(content::PresentationError(
841 content::PRESENTATION_ERROR_SESSION_REQUEST_CANCELLED, 842 content::PRESENTATION_ERROR_SESSION_REQUEST_CANCELLED,
842 "Auto-join request cancelled by user preferences.")); 843 "Auto-join request cancelled by user preferences."));
843 return; 844 return;
844 } 845 }
845 #endif // !defined(OS_ANDROID) 846 #endif // !defined(OS_ANDROID)
846 847
847 // TODO(crbug.com/627655): Handle multiple URLs. 848 // TODO(crbug.com/627655): Handle multiple URLs.
848 const GURL& presentation_url = presentation_urls[0]; 849 const GURL& presentation_url = presentation_urls[0];
849 bool incognito = web_contents_->GetBrowserContext()->IsOffTheRecord(); 850 bool incognito = web_contents_->GetBrowserContext()->IsOffTheRecord();
850 std::vector<MediaRouteResponseCallback> route_response_callbacks; 851 std::vector<MediaRouteResponseCallback> route_response_callbacks;
851 route_response_callbacks.push_back( 852 route_response_callbacks.push_back(
852 base::Bind(&PresentationServiceDelegateImpl::OnJoinRouteResponse, 853 base::Bind(&PresentationServiceDelegateImpl::OnJoinRouteResponse,
853 weak_factory_.GetWeakPtr(), render_process_id, render_frame_id, 854 weak_factory_.GetWeakPtr(), render_process_id, render_frame_id,
854 presentation_url, presentation_id, success_cb, error_cb)); 855 presentation_url, presentation_id, success_cb, error_cb));
855 router_->JoinRoute(MediaSourceForPresentationUrl(presentation_url).id(), 856 router_->JoinRoute(MediaSourceForPresentationUrl(presentation_url).id(),
856 presentation_id, origin.GetURL(), web_contents_, 857 presentation_id, origin, web_contents_,
857 route_response_callbacks, base::TimeDelta(), incognito); 858 route_response_callbacks, base::TimeDelta(), incognito);
858 } 859 }
859 860
860 void PresentationServiceDelegateImpl::CloseConnection( 861 void PresentationServiceDelegateImpl::CloseConnection(
861 int render_process_id, 862 int render_process_id,
862 int render_frame_id, 863 int render_frame_id,
863 const std::string& presentation_id) { 864 const std::string& presentation_id) {
864 const RenderFrameHostId rfh_id(render_process_id, render_frame_id); 865 const RenderFrameHostId rfh_id(render_process_id, render_frame_id);
865 const MediaRoute::Id& route_id = 866 const MediaRoute::Id& route_id =
866 frame_manager_->GetRouteId(rfh_id, presentation_id); 867 frame_manager_->GetRouteId(rfh_id, presentation_id);
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
996 const base::ListValue* origins = 997 const base::ListValue* origins =
997 Profile::FromBrowserContext(web_contents_->GetBrowserContext()) 998 Profile::FromBrowserContext(web_contents_->GetBrowserContext())
998 ->GetPrefs() 999 ->GetPrefs()
999 ->GetList(prefs::kMediaRouterTabMirroringSources); 1000 ->GetList(prefs::kMediaRouterTabMirroringSources);
1000 return origins && 1001 return origins &&
1001 origins->Find(base::StringValue(origin.Serialize())) != origins->end(); 1002 origins->Find(base::StringValue(origin.Serialize())) != origins->end();
1002 } 1003 }
1003 #endif // !defined(OS_ANDROID) 1004 #endif // !defined(OS_ANDROID)
1004 1005
1005 } // namespace media_router 1006 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698