[Presentation API] (alternative) 1-UA: send message between controller and receiver page

content/renderer/presentation/presentation_connection_client.h

 // 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.
 
 #ifndef CONTENT_RENDERER_PRESENTATION_PRESENTATION_SESSION_CLIENT_H_
 #define CONTENT_RENDERER_PRESENTATION_PRESENTATION_SESSION_CLIENT_H_
 
 #include "base/compiler_specific.h"
 #include "content/common/content_export.h"
 #include "third_party/WebKit/public/platform/modules/presentation/WebPresentationConnectionClient.h"
 #include "third_party/WebKit/public/platform/modules/presentation/presentation.mojom.h"
 #include "url/gurl.h"
 
 namespace content {
 
 // PresentationConnectionClient is passed to the Blink layer if presentation
 // session has been created successfully. Owned by the callback.
 class CONTENT_EXPORT PresentationConnectionClient
     : public NON_EXPORTED_BASE(blink::WebPresentationConnectionClient) {
  public:
   explicit PresentationConnectionClient(
       blink::mojom::PresentationSessionInfoPtr session_info);
-  explicit PresentationConnectionClient(const GURL& url,
-                                        const mojo::String& id);
+  PresentationConnectionClient(const GURL& url, const mojo::String& id);
+  // Call this constructor to create PresentationConnectionClient for
+  // 1-UA presentation.
+  // |session_info|: contains presentation id and url info

[mark a. foltz, 2016/10/21 00:53:02]

nit: "Contains presentation ID and URL info."

[zhaobin, 2016/10/22 02:44:13]

Done.

+  // |proxy|: proxy to blink PresentationConnection object in current render

[mark a. foltz, 2016/10/21 00:53:02]

nit: Proxy

[zhaobin, 2016/10/22 02:44:13]

Done.

+  //          process. Ownership is passed to this class.
+  PresentationConnectionClient(
+      blink::mojom::PresentationSessionInfoPtr session_info,
+      std::unique_ptr<blink::WebPresentationConnectionProxy> proxy);
   ~PresentationConnectionClient() override;
 
   // WebPresentationConnectionClient implementation.
   blink::WebURL getUrl() override;
   blink::WebString getId() override;
+  // Returns proxy passed into this class.
+  // It will transfer ownership of |proxy_| to calling object
+  // (PresentationConnection).

[mark a. foltz, 2016/10/21 00:53:02]

Rewrap comments at 80 columns.

[zhaobin, 2016/10/22 02:44:13]

Done.

+  std::unique_ptr<blink::WebPresentationConnectionProxy> getProxy() override;

[mark a. foltz, 2016/10/21 00:53:02]

Can this be takeProxy() to denote change of ownership?

[zhaobin, 2016/10/22 02:44:13]

Done.

 
  private:
   blink::WebURL url_;
   blink::WebString id_;
+  std::unique_ptr<blink::WebPresentationConnectionProxy> proxy_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_PRESENTATION_PRESENTATION_SESSION_CLIENT_H_