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

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

Issue 2264153002: [Presentation API] Add support for multiple URLs in PresentationRequest on Media Router UI side (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address Mark's comments, add unit test for CMWMS Created 4 years, 3 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/create_presentation_connection_request.cc
diff --git a/chrome/browser/media/router/create_presentation_connection_request.cc b/chrome/browser/media/router/create_presentation_connection_request.cc
index d96d1915b9ef1238ef89c197a600036de5675c7c..4ac9bb13203ec00746155fc804509c5a33afa60b 100644
--- a/chrome/browser/media/router/create_presentation_connection_request.cc
+++ b/chrome/browser/media/router/create_presentation_connection_request.cc
@@ -4,6 +4,8 @@
#include "chrome/browser/media/router/create_presentation_connection_request.h"
+#include <memory>
mark a. foltz 2016/09/02 23:00:37 This should go in the .h
takumif 2016/09/06 21:53:20 Done.
+
#include "chrome/browser/media/router/media_source_helper.h"
#include "chrome/browser/media/router/route_request_result.h"
@@ -18,7 +20,8 @@ CreatePresentationConnectionRequest::CreatePresentationConnectionRequest(
const GURL& frame_url,
const PresentationSessionSuccessCallback& success_cb,
const PresentationSessionErrorCallback& error_cb)
- : presentation_request_(render_frame_host_id, presentation_url, frame_url),
+ : presentation_request_(
+ render_frame_host_id, {presentation_url}, frame_url),
success_cb_(success_cb),
error_cb_(error_cb),
cb_invoked_(false) {

Powered by Google App Engine
This is Rietveld 408576698