Non-Local Join for Media Router and Presentation API

chrome/browser/media/android/router/media_router_dialog_controller_android.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 CHROME_BROWSER_MEDIA_ANDROID_ROUTER_MEDIA_ROUTER_DIALOG_CONTROLLER_ANDROID_H_
 #define CHROME_BROWSER_MEDIA_ANDROID_ROUTER_MEDIA_ROUTER_DIALOG_CONTROLLER_ANDROID_H_
 
 #include <jni.h>
 
 #include "base/android/scoped_java_ref.h"
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/media/router/media_router_dialog_controller.h"
 #include "chrome/browser/media/router/media_routes_observer.h"
 #include "content/public/browser/web_contents_user_data.h"
 
 namespace media_router {
 
 // Android implementation of the MediaRouterDialogController.
 class MediaRouterDialogControllerAndroid
     : public content::WebContentsUserData<MediaRouterDialogControllerAndroid>,
-      public MediaRouterDialogController,
-      public MediaRoutesObserver {
+      public MediaRouterDialogController {
  public:
   ~MediaRouterDialogControllerAndroid() override;
 
   static bool Register(JNIEnv* env);
 
   static MediaRouterDialogControllerAndroid* GetOrCreateForWebContents(
       content::WebContents* web_contents);
 
   // The methods called by the Java counterpart.
 
@@ skipping 11 matching lines @@
   // Use MediaRouterDialogControllerAndroid::CreateForWebContents() to create an
   // instance.
   explicit MediaRouterDialogControllerAndroid(
       content::WebContents* web_contents);
 
   // MediaRouterDialogController:
   void CreateMediaRouterDialog() override;
   void CloseMediaRouterDialog() override;
   bool IsShowingMediaRouterDialog() const override;
 
-  // MediaRoutesObserver:
-  void OnRoutesUpdated(const std::vector<MediaRoute>& routes) override;
+  class UIMediaRoutesObserver : public MediaRoutesObserver {
+   public:
+    using RoutesUpdatedCallback =
+        base::Callback<void(const std::vector<MediaRoute>&,
+            const std::vector<MediaRoute::Id>&)>;
+    UIMediaRoutesObserver(MediaRouter* router, const MediaSource::Id source_id,
+                          const RoutesUpdatedCallback& callback);
+    ~UIMediaRoutesObserver() override;
+
+    // MediaRoutesObserver
+    void OnRoutesUpdated(
+        const std::vector<MediaRoute>& routes,
+        const std::vector<MediaRoute::Id>& joinable_route_ids) override;
+
+   private:
+    // Callback to the owning MediaRouterUI instance.

[whywhat, 2015/11/26 14:43:04]

We don't have MediaRouterUI on Android.

[matt.boetger, 2015/12/01 01:26:55]

Reverting.

+    RoutesUpdatedCallback callback_;
+
+    DISALLOW_COPY_AND_ASSIGN(UIMediaRoutesObserver);
+  };
 
   void CancelPresentationRequest();
 
+  void OnRoutesUpdated(
+      const std::vector<MediaRoute>& routes,
+      const std::vector<MediaRoute::Id>& joinable_route_ids);
+
   base::android::ScopedJavaGlobalRef<jobject> java_dialog_controller_;
 
   // Null if no routes or more than one route exist. If there's only one route,
   // keeps a copy to determine if the route controller dialog needs to be shown
   // vs. the route chooser one.
   scoped_ptr<MediaRoute> single_existing_route_;
 
+  // The routes_observer_ only needs to be active when the dialog is open.
+  scoped_ptr<MediaRoutesObserver> routes_observer_;
+
+  // Pointer to the MediaRouter for this instance's BrowserContext.
+  MediaRouter* router_;

[imcheng, 2015/11/26 00:49:45]

MediaRouter* const router_;

[matt.boetger, 2015/12/01 01:26:55]

Reverting.

+
   DISALLOW_COPY_AND_ASSIGN(MediaRouterDialogControllerAndroid);
 };
 
 }  // namespace media_router
 
 #endif  // CHROME_BROWSER_MEDIA_ANDROID_ROUTER_MEDIA_ROUTER_DIALOG_CONTROLLER_ANDROID_H_