[PresentationAPI] Added plumbing for start/joinSession from JS to platform

Source/modules/presentation/PresentationController.cpp

 // 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 "config.h"
 #include "modules/presentation/PresentationController.h"
 
 #include "core/frame/LocalFrame.h"
 #include "public/platform/modules/presentation/WebPresentationClient.h"
 
@@ skipping 56 matching lines @@
         return false;
     return m_presentation->isAvailableChangeWatched();
 }
 
 void PresentationController::updateAvailableChangeWatched(bool watched)
 {
     if (m_client)
         m_client->updateAvailableChangeWatched(watched);
 }
 
+void PresentationController::startSession(const String& presentationUrl, const String& presentationId, WebPresentationSessionClientCallbacks* callbacks)
+{
+    if (m_client)
+        m_client->startSession(presentationUrl, presentationId, callbacks);

[mlamouri (slow - plz ping), 2015/02/25 20:51:06]

Hmm, sorry for realizing that now, but you have to |delete callbacks;| if there
is no |m_client|, otherwise, you are leaking that.

Maybe something like:
if (!m_client) {
  delete callbacks;
  return;
}
m_client->...

[whywhat, 2015/02/25 21:04:14]

I start thinking I need a ref-counted pointer here...

[mlamouri (slow - plz ping), 2015/02/25 21:08:54]

There is no ref pointer that crosses Blink/Chromium boundary.

[whywhat, 2015/02/25 22:46:51]

Something simple like std::unique_ptr would work in this case. It seems Blink
lacks such a pointer and RefPtr is too much :(

+}
+
+void PresentationController::joinSession(const String& presentationUrl, const String& presentationId, WebPresentationSessionClientCallbacks* callbacks)
+{
+    if (m_client)
+        m_client->joinSession(presentationUrl, presentationId, callbacks);

[mlamouri (slow - plz ping), 2015/02/25 20:51:06]

ditto

[whywhat, 2015/02/25 22:46:52]

Done.

+}
+
 void PresentationController::setPresentation(Presentation* presentation)
 {
     m_presentation = presentation;
 }
 
 void PresentationController::willDetachFrameHost()
 {
     if (m_client) {
         m_client->setController(nullptr);
         m_client = nullptr;
     }
 }
 
 } // namespace blink