Activate WebRTC data channel tests for WK1
Created attachment 306765 [details] Patch
Comment on attachment 306765 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=306765&action=review > Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:67 > + ASSERT(client.factory()); Won't the m_peerConnectionFactory assignment above have crashed already if the factory is NULL?
Created attachment 306826 [details] Patch for landing
(In reply to Eric Carlson from comment #2) > Comment on attachment 306765 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=306765&action=review > > > Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:67 > > + ASSERT(client.factory()); > > Won't the m_peerConnectionFactory assignment above have crashed already if > the factory is NULL? This is undefined behavior so it should be fine but we should not do that. I removed the ASSERT. Hopefully, we can make factory() returning a reference now. I'll do this in a follow-up
Comment on attachment 306826 [details] Patch for landing Clearing flags on attachment: 306826 Committed r215239: <http://trac.webkit.org/changeset/215239>
All reviewed patches have been landed. Closing bug.