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

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

Issue 2962623002: [Media Router] Clean up OffscreenPresentationManager params. (Closed)
Patch Set: Addressed Bin's comments Created 3 years, 6 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 58ac00d771cc5c0b59eb891feb5595fd7ecf8f9b..1b74e688b41420edc5f26b47adfbfdda8a386b21 100644
--- a/chrome/browser/media/router/presentation_service_delegate_impl.cc
+++ b/chrome/browser/media/router/presentation_service_delegate_impl.cc
@@ -289,8 +289,8 @@ void PresentationFrame::ConnectToPresentation(
OffscreenPresentationManagerFactory::GetOrCreateForWebContents(
web_contents_);
offscreen_presentation_manager->RegisterOffscreenPresentationController(
- presentation_info.presentation_id, presentation_info.presentation_url,
- render_frame_host_id_, std::move(controller_connection_ptr),
+ presentation_info, render_frame_host_id_,
+ std::move(controller_connection_ptr),
std::move(receiver_connection_request), pid_route_it->second);
} else {
DVLOG(2)

Powered by Google App Engine
This is Rietveld 408576698