[Presentation API] 1-UA: notify receiver page when receiver connection becomes available (blink sid…

third_party/WebKit/Source/modules/presentation/Presentation.cpp

Index: third_party/WebKit/Source/modules/presentation/Presentation.cpp
diff --git a/third_party/WebKit/Source/modules/presentation/Presentation.cpp b/third_party/WebKit/Source/modules/presentation/Presentation.cpp
index d20a52919ae54a9f4886c00058428d9b6c77c0a1..649a4d4e8065c0f636bbb613c9661fd56f2305e7 100644
--- a/third_party/WebKit/Source/modules/presentation/Presentation.cpp
+++ b/third_party/WebKit/Source/modules/presentation/Presentation.cpp
@@ -56,11 +56,13 @@ void Presentation::setDefaultRequest(PresentationRequest* request)
 
 PresentationReceiver* Presentation::receiver()
 {
+    PresentationController* controller = PresentationController::from(*frame());
+    auto client = controller ? controller->client() : nullptr;

[mark a. foltz, 2016/09/15 16:53:18]

It looks like this can only be nullptr if Presentation API is disabled as a
runtime feature.

It would clean up a lot of these extra checks if we could assume it was always
enabled.   Can we assume that now Mounir?

[zhaobin, 2016/09/15 18:56:03]

Acknowledged.

     // TODO(mlamouri): only return something if the Blink instance is running in
     // presentation receiver mode. The flag PresentationReceiver could be used
     // for that.

[mark a. foltz, 2016/09/15 16:53:18]

We need to actually do this to match spec; navigator.presentation.receiver must
be null in a non-receiving browsing context.

http://w3c.github.io/presentation-api/#receiving-user-agent

Can you replace TODO(mlamouri) here with TODO(crbug.com/647296) here?

[zhaobin, 2016/09/15 18:56:03]

Done.

     if (!m_receiver)
-        m_receiver = new PresentationReceiver(frame());
+        m_receiver = new PresentationReceiver(frame(), client);
     return m_receiver;
 }