Hook up RendererMediaSessionManager with browser side

content/renderer/media/android/webmediasession_android.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/media/android/webmediasession_android.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "content/renderer/media/android/renderer_media_session_manager.h"
 
 namespace content {
 
 WebMediaSessionAndroid::WebMediaSessionAndroid(
     RendererMediaSessionManager* session_manager)
-    : session_manager_(session_manager) {
+    : session_manager_(session_manager), next_request_id_(0) {
   DCHECK(session_manager_);
   session_id_ = session_manager_->RegisterMediaSession(this);
 }
 
 WebMediaSessionAndroid::~WebMediaSessionAndroid() {
   session_manager_->UnregisterMediaSession(session_id_);
 }
 
 void WebMediaSessionAndroid::activate(
-    blink::WebMediaSessionActivateCallback* raw_callback) {
-  NOTIMPLEMENTED();
-
-  scoped_ptr<blink::WebMediaSessionActivateCallback> callback(raw_callback);
-  callback->onError(blink::WebMediaSessionError::Activate);
+    blink::WebMediaSessionActivateCallback* callback) {
+  int request_id = activation_requests_.Add(callback);
+  session_manager_->Activate(session_id_, request_id);
 }
 
 void WebMediaSessionAndroid::deactivate(
-    blink::WebMediaSessionDeactivateCallback* raw_callback) {
-  NOTIMPLEMENTED();
+    blink::WebMediaSessionDeactivateCallback* callback) {
+  int request_id = deactivation_requests_.Add(callback);
+  session_manager_->Deactivate(session_id_, request_id);
+}
 
-  scoped_ptr<blink::WebMediaSessionDeactivateCallback> callback(raw_callback);
+bool WebMediaSessionAndroid::hasPendingCallbacks() {
+  return !activation_requests_.IsEmpty() || !deactivation_requests_.IsEmpty();
+}
+
+void WebMediaSessionAndroid::OnDidActivate(int request_id, bool success) {
+  blink::WebMediaSessionActivateCallback* callback =
+      activation_requests_.Lookup(request_id);
+  if (!callback)
+    return;

[mlamouri (slow - plz ping), 2015/12/07 18:14:52]

Should you DCHECK()? It seems that this should never happen.

[davve, 2015/12/08 10:21:23]

Depends on what we want to test, I guess.

In
https://codereview.chromium.org/1441883003/diff/80001/content/renderer/media/...
we added tests that passes unknown and unexpected *request_ids*. For symmetry, I
added similar testing for *session_id* in ps#8.

If we want such tests, we need to handle it run-time somehow, and not DCHECK it.
(Or make the testharness expect the DCHECK to trigger. Don't know anything about
that feasibility).

Happy to take advice on general stance here.

+
+  if (success)
+    callback->onSuccess();
+  else
+    callback->onError(
+        blink::WebMediaSessionError(blink::WebMediaSessionError::Activate));
+  activation_requests_.Remove(request_id);
+}
+
+void WebMediaSessionAndroid::OnDidDeactivate(int request_id) {
+  blink::WebMediaSessionDeactivateCallback* callback =
+      deactivation_requests_.Lookup(request_id);
+  if (!callback)
+    return;

[mlamouri (slow - plz ping), 2015/12/07 18:14:52]

ditto

[davve, 2015/12/08 10:21:23]

See other reply.

+
   callback->onSuccess();
+  deactivation_requests_.Remove(request_id);
 }
 
 }  // namespace content