[Presentation API] (4th)(1-UA blink side) Add WebPresentationConnection and WebPresentationConnecti…

content/renderer/presentation/presentation_connection_client.cc

 // Copyright 2015 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 "content/renderer/presentation/presentation_connection_client.h"
 
 #include "base/logging.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
+#include "third_party/WebKit/public/platform/modules/presentation/WebPresentationConnectionProxy.h"
 
 namespace content {
 
 PresentationConnectionClient::PresentationConnectionClient(
     blink::mojom::PresentationSessionInfoPtr session_info)
-    : url_(session_info->url),
-      id_(blink::WebString::fromUTF8(session_info->id)) {}
+    : PresentationConnectionClient(std::move(session_info), nullptr) {}

[mark a. foltz, 2016/11/16 01:16:39]

If this object is taking ownership of |session_info|, it's a little odd for it
to copy the contents and immediately destroy it after the ctor finishes.  I
would have expected it to keep |session_info| as a class member.

I guess that is okay though, it's an implementation detail what this class does
with it.

[zhaobin, 2016/11/23 22:52:29]

Done.

 
 PresentationConnectionClient::PresentationConnectionClient(
-    const GURL& url,
-    const mojo::String& id)
-    : url_(url),
-      id_(blink::WebString::fromUTF8(id)) {}
+    blink::mojom::PresentationSessionInfoPtr session_info,
+    std::unique_ptr<blink::WebPresentationConnectionProxy> proxy)
+    : url_(session_info->url),
+      id_(blink::WebString::fromUTF8(session_info->id)),
+      proxy_(std::move(proxy)) {}
 
 PresentationConnectionClient::~PresentationConnectionClient() {
 }
 
 blink::WebURL PresentationConnectionClient::getUrl() {
     return url_;
 }
 
 blink::WebString PresentationConnectionClient::getId() {
     return id_;
 }
 
+std::unique_ptr<blink::WebPresentationConnectionProxy>
+PresentationConnectionClient::takeProxy() {
+  return std::move(proxy_);
+}
+
 }  // namespace content