diff --git a/api/peerconnectioninterface.h b/api/peerconnectioninterface.h index eb6685a561..6293361afd 100644 --- a/api/peerconnectioninterface.h +++ b/api/peerconnectioninterface.h @@ -973,6 +973,8 @@ class PeerConnectionObserver { kIceState, }; + virtual ~PeerConnectionObserver() = default; + // Triggered when the SignalingState changed. virtual void OnSignalingChange( PeerConnectionInterface::SignalingState new_state) = 0; @@ -1045,10 +1047,6 @@ class PeerConnectionObserver { // TODO(hbos,deadbeef): Make pure virtual when all subclasses implement it. virtual void OnRemoveTrack( rtc::scoped_refptr receiver) {} - - protected: - // Dtor protected as objects shouldn't be deleted via this interface. - ~PeerConnectionObserver() {} }; // PeerConnectionFactoryInterface is the factory interface used for creating diff --git a/sdk/android/src/jni/pc/peerconnection.cc b/sdk/android/src/jni/pc/peerconnection.cc index ed1598034a..d904f07d3c 100644 --- a/sdk/android/src/jni/pc/peerconnection.cc +++ b/sdk/android/src/jni/pc/peerconnection.cc @@ -329,8 +329,7 @@ static void JNI_PeerConnection_FreePeerConnectionObserver( JNIEnv*, const JavaParamRef&, jlong j_p) { - PeerConnectionObserverJni* p = - reinterpret_cast(j_p); + PeerConnectionObserver* p = reinterpret_cast(j_p); delete p; }