Separating internal and external methods of RtpSender/RtpReceiver.

webrtc/api/proxy.h

 /*
  *  Copyright 2013 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 288 matching lines @@
   C* c_;
   Method m_;
   ReturnType<R> r_;
   T1 a1_;
   T2 a2_;
   T3 a3_;
   T4 a4_;
   T5 a5_;
 };
 
-#define BEGIN_SIGNALING_PROXY_MAP(c)                                     \
-  class c##Proxy : public c##Interface {                                  \
+#define BEGIN_SIGNALING_PROXY_MAP(c)                                      \
+  template <class INNER_CLASS>                             \
+  class c##ProxyEx;                                  \

[Taylor Brandstetter, 2016/06/01 20:04:26]

I know that "ProxyEx" is a terrible name; what do you suggest? "ProxyTo<Thing>"?
"ProxyWithInner<Thing>"?

+  typedef c##ProxyEx<c##Interface> c##Proxy; \
+  template <class INNER_CLASS>                             \
+  class c##ProxyEx : public c##Interface {                                  \
    protected:                                                             \
     typedef c##Interface C;                                               \
-    c##Proxy(rtc::Thread* signaling_thread, C* c)                         \
+    c##ProxyEx(rtc::Thread* signaling_thread, INNER_CLASS* c)               \
       : signaling_thread_(signaling_thread), c_(c) {}                     \
-    ~c##Proxy() {                                                         \
-      MethodCall0<c##Proxy, void> call(                                   \
-          this, &c##Proxy::Release_s);                                    \
+    ~c##ProxyEx() {                                                         \
+      MethodCall0<c##ProxyEx, void> call(                                   \
+          this, &c##ProxyEx::Release_s);                                    \
       call.Marshal(signaling_thread_);                                    \
     }                                                                     \
                                                                           \
    public:                                                                \
-    static rtc::scoped_refptr<C> Create(rtc::Thread* signaling_thread, C* c) { \
-      return new rtc::RefCountedObject<c##Proxy>(                         \
+    static rtc::scoped_refptr<c##ProxyEx> Create(rtc::Thread* signaling_thread, INNER_CLASS* c) { \
+      return new rtc::RefCountedObject<c##ProxyEx>(                         \
           signaling_thread, c);                                           \
-    }
+    }                                                                     \
+    const INNER_CLASS* get() const { return c_.get(); }                   \
+    INNER_CLASS* get() { return c_.get(); }                               \
 
 #define BEGIN_PROXY_MAP(c)                                              \
-  class c##Proxy : public c##Interface {                                \
+  template <class INNER_CLASS>                             \
+  class c##ProxyEx;                                  \
+  typedef c##ProxyEx<c##Interface> c##Proxy; \
+  template <class INNER_CLASS>                             \
+  class c##ProxyEx : public c##Interface {                                  \
    protected:                                                           \
     typedef c##Interface C;                                             \
-    c##Proxy(rtc::Thread* signaling_thread, rtc::Thread* worker_thread, C* c) \
+    c##ProxyEx(rtc::Thread* signaling_thread, rtc::Thread* worker_thread, \
+             INNER_CLASS* c)                                            \
       : signaling_thread_(signaling_thread),                            \
         worker_thread_(worker_thread),                                  \
         c_(c) {}                                                        \
-    ~c##Proxy() {                                                       \
-      MethodCall0<c##Proxy, void> call(this, &c##Proxy::Release_s);     \
+    ~c##ProxyEx() {                                                       \
+      MethodCall0<c##ProxyEx, void> call(this, &c##ProxyEx::Release_s);     \
       call.Marshal(signaling_thread_);                                  \
     }                                                                   \
                                                                         \
    public:                                                              \
-    static rtc::scoped_refptr<C> Create(                                \
-        rtc::Thread* signaling_thread, rtc::Thread* worker_thread, C* c) { \
-      return new rtc::RefCountedObject<c##Proxy>(                       \
+    static rtc::scoped_refptr<c##ProxyEx> Create(                                \
+        rtc::Thread* signaling_thread, rtc::Thread* worker_thread, INNER_CLASS* c) { \
+      return new rtc::RefCountedObject<c##ProxyEx>(                       \
           signaling_thread, worker_thread, c);                          \
-    }
+    }                                                                   \
+    const INNER_CLASS* get() const { return c_.get(); }                 \
+    INNER_CLASS* get() { return c_.get(); }                             \
 
 #define PROXY_METHOD0(r, method)                  \
   r method() override {                           \
     MethodCall0<C, r> call(c_.get(), &C::method); \
     return call.Marshal(signaling_thread_);       \
   }
 
 #define PROXY_CONSTMETHOD0(r, method)                  \
   r method() const override {                          \
     ConstMethodCall0<C, r> call(c_.get(), &C::method); \
@@ skipping 50 matching lines @@
     MethodCall2<C, r, t1, t2> call(c_.get(), &C::method, a1, a2); \
     return call.Marshal(worker_thread_);                          \
   }
 
 #define END_SIGNALING_PROXY() \
    private:\
     void Release_s() {\
       c_ = NULL;\
     }\
     mutable rtc::Thread* signaling_thread_;\
-    rtc::scoped_refptr<C> c_;\
+    rtc::scoped_refptr<INNER_CLASS> c_;\
   };
 
 #define END_PROXY()                                  \
    private:                                          \
     void Release_s() {                               \
       c_ = NULL;                                     \
     }                                                \
     mutable rtc::Thread* signaling_thread_;          \
     mutable rtc::Thread* worker_thread_;             \
-    rtc::scoped_refptr<C> c_;                        \
+    rtc::scoped_refptr<INNER_CLASS> c_;              \
   };                                                 \
 
 }  // namespace webrtc
 
 #endif  //  WEBRTC_API_PROXY_H_