| Index: chrome/browser/media/router/presentation_service_delegate_impl.cc
|
| diff --git a/chrome/browser/media/router/presentation_service_delegate_impl.cc b/chrome/browser/media/router/presentation_service_delegate_impl.cc
|
| index 2e4bfe772e572a97d90523381fcc08ac68327b53..ea2dc3ab93046a411d66b7f3566e017628a9b0e2 100644
|
| --- a/chrome/browser/media/router/presentation_service_delegate_impl.cc
|
| +++ b/chrome/browser/media/router/presentation_service_delegate_impl.cc
|
| @@ -60,12 +60,14 @@
|
|
|
| // Gets the last committed URL for the render frame specified by
|
| // |render_frame_host_id|.
|
| -url::Origin GetLastCommittedURLForFrame(
|
| - RenderFrameHostId render_frame_host_id) {
|
| +GURL GetLastCommittedURLForFrame(RenderFrameHostId render_frame_host_id) {
|
| RenderFrameHost* render_frame_host = RenderFrameHost::FromID(
|
| render_frame_host_id.first, render_frame_host_id.second);
|
| - DCHECK(render_frame_host);
|
| - return render_frame_host->GetLastCommittedOrigin();
|
| + if (!render_frame_host)
|
| + return GURL();
|
| +
|
| + // TODO(crbug.com/632623): Use url::Origin in place of GURL for origins
|
| + return render_frame_host->GetLastCommittedOrigin().GetURL();
|
| }
|
|
|
| // Observes messages originating from the MediaSink connected to a MediaRoute
|
| @@ -221,7 +223,7 @@
|
|
|
| sinks_observer.reset(new PresentationMediaSinksObserver(
|
| router_, listener, source,
|
| - GetLastCommittedURLForFrame(render_frame_host_id_)));
|
| + GetLastCommittedURLForFrame(render_frame_host_id_).GetOrigin()));
|
|
|
| if (!sinks_observer->Init()) {
|
| url_to_sinks_observer_.erase(source.id());
|
| @@ -531,10 +533,9 @@
|
| ClearDefaultPresentationRequest();
|
| } else {
|
| DCHECK(!callback.is_null());
|
| - const auto& frame_origin =
|
| - GetLastCommittedURLForFrame(render_frame_host_id);
|
| + GURL frame_url(GetLastCommittedURLForFrame(render_frame_host_id));
|
| PresentationRequest request(render_frame_host_id, default_presentation_urls,
|
| - frame_origin);
|
| + frame_url);
|
| default_presentation_started_callback_ = callback;
|
| SetDefaultPresentationRequest(request);
|
| }
|
| @@ -787,8 +788,8 @@
|
| return;
|
| }
|
|
|
| - const url::Origin& origin = GetLastCommittedURLForFrame(
|
| - RenderFrameHostId(render_process_id, render_frame_id));
|
| + const url::Origin& origin = url::Origin(GetLastCommittedURLForFrame(
|
| + RenderFrameHostId(render_process_id, render_frame_id)));
|
|
|
| #if !defined(OS_ANDROID)
|
| if (IsAutoJoinPresentationId(presentation_id) &&
|
| @@ -809,7 +810,7 @@
|
| weak_factory_.GetWeakPtr(), render_process_id, render_frame_id,
|
| presentation_url, presentation_id, success_cb, error_cb));
|
| router_->JoinRoute(MediaSourceForPresentationUrl(presentation_url).id(),
|
| - presentation_id, origin, web_contents_,
|
| + presentation_id, origin.GetURL(), web_contents_,
|
| route_response_callbacks, base::TimeDelta(), incognito);
|
| }
|
|
|
|
|