[Media Router] Handle multiple Presentation URLs when creating routes

chrome/browser/media/router/route_request_result.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/media/router/route_request_result.h"
 
 #include "base/memory/ptr_util.h"
 #include "chrome/browser/media/router/media_route.h"
 
 namespace media_router {
@@ skipping 15 matching lines @@
       new RouteRequestResult(nullptr, "", error, result_code));
 }
 
 RouteRequestResult::RouteRequestResult(std::unique_ptr<MediaRoute> route,
                                        const std::string& presentation_id,
                                        const std::string& error,
                                        ResultCode result_code)
     : route_(std::move(route)),
       presentation_id_(presentation_id),
       error_(error),
-      result_code_(result_code) {}
+      result_code_(result_code) {
+  presentation_url_ = route_ ? GURL(route_->media_source().id()) : GURL("");

[mark a. foltz, 2016/12/02 20:59:09]

There is a TODO to have MediaSource contain a GURL if it was created from a URL.
 That would make this cleaner, but not necessary to fix in this patch.

[zhaobin, 2016/12/02 22:50:57]

Done.

+}
 
 RouteRequestResult::~RouteRequestResult() = default;
 
 }  // namespace media_router