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

Unified Diff: chrome/browser/media/router/presentation_service_delegate_impl.cc

Issue 2433283005: [Media Router] Use origin instead of URL for frame. (Closed)
Patch Set: Remove DCHECK in media_router_ui.cc Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
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 0a40a5b3346a2fcb5d13b3d9513211d0e0366b9e..d0c574cfd30e5b0d9beee443660035d56d5e9618 100644
--- a/chrome/browser/media/router/presentation_service_delegate_impl.cc
+++ b/chrome/browser/media/router/presentation_service_delegate_impl.cc
@@ -55,7 +55,10 @@ RenderFrameHostId GetRenderFrameHostId(RenderFrameHost* render_frame_host) {
GURL GetLastCommittedURLForFrame(RenderFrameHostId render_frame_host_id) {
RenderFrameHost* render_frame_host = RenderFrameHost::FromID(
render_frame_host_id.first, render_frame_host_id.second);
- return render_frame_host ? render_frame_host->GetLastCommittedURL() : GURL();
+ // TODO(crbug.com/632623): Use url::Origin in place of GURL for origins
+ return render_frame_host
+ ? render_frame_host->GetLastCommittedOrigin().GetURL()
+ : GURL();
}
// Observes messages originating from the MediaSink connected to a MediaRoute
« no previous file with comments | « chrome/browser/media/router/presentation_request.h ('k') | chrome/browser/ui/webui/media_router/media_router_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698