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

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

Issue 1202963004: Gets presentation ID from route ID upon route creation, and overrides previous presentation ID with… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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/media_router_mojo_impl.cc
diff --git a/chrome/browser/media/router/media_router_mojo_impl.cc b/chrome/browser/media/router/media_router_mojo_impl.cc
index aab17bb30bed8e07cfe4b19cc903858bf521f0c8..c9c57fed476216831bcd3c538303ee3ead7b33ab 100644
--- a/chrome/browser/media/router/media_router_mojo_impl.cc
+++ b/chrome/browser/media/router/media_router_mojo_impl.cc
@@ -49,9 +49,9 @@ void EventPageWakeComplete(bool success) {
scoped_ptr<content::PresentationSessionMessage>
ConvertToPresentationSessionMessage(interfaces::RouteMessagePtr input) {
DCHECK(!input.is_null());
- // TODO(haibinlu): get presentation_url&id from route_id
- std::string presentation_url;
- std::string presentation_id;
+ const auto& id_and_url = GetPresentationIdAndUrl(input->route_id);
+ const std::string& presentation_id = id_and_url.first;
+ const std::string& presentation_url = id_and_url.second;
scoped_ptr<content::PresentationSessionMessage> output;
switch (input->type) {
case interfaces::RouteMessage::Type::TYPE_TEXT: {

Powered by Google App Engine
This is Rietveld 408576698