WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
199177
Enable a debug WebRTC mode without any encryption
https://bugs.webkit.org/show_bug.cgi?id=199177
Summary
Enable a debug WebRTC mode without any encryption
youenn fablet
Reported
2019-06-24 15:38:34 PDT
Enable a debug WebRTC mode without any encryption
Attachments
Patch
(11.61 KB, patch)
2019-06-28 14:45 PDT
,
youenn fablet
no flags
Details
Formatted Diff
Diff
Archive of layout-test-results from ews104 for mac-highsierra-wk2
(2.76 MB, application/zip)
2019-06-28 16:07 PDT
,
EWS Watchlist
no flags
Details
Archive of layout-test-results from ews122 for ios-simulator-wk2
(2.66 MB, application/zip)
2019-06-28 16:51 PDT
,
EWS Watchlist
no flags
Details
Patch
(11.63 KB, patch)
2019-07-07 13:54 PDT
,
youenn fablet
no flags
Details
Formatted Diff
Diff
Patch for landing
(18.63 KB, patch)
2019-07-08 10:54 PDT
,
youenn fablet
no flags
Details
Formatted Diff
Diff
Show Obsolete
(4)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2019-06-24 15:39:19 PDT
<
rdar://problem/52074986
>
youenn fablet
Comment 2
2019-06-24 15:41:43 PDT
This was requested to easily debug, for instance video codec issues. This can be done using PeerConnectionFactoryInterface::Options::disable_encryption
youenn fablet
Comment 3
2019-06-28 14:45:53 PDT
Created
attachment 373151
[details]
Patch
EWS Watchlist
Comment 4
2019-06-28 16:07:48 PDT
Comment on
attachment 373151
[details]
Patch
Attachment 373151
[details]
did not pass mac-wk2-ews (mac-wk2): Output:
https://webkit-queues.webkit.org/results/12607036
New failing tests: webrtc/pc-detached-document.html
EWS Watchlist
Comment 5
2019-06-28 16:07:49 PDT
Created
attachment 373159
[details]
Archive of layout-test-results from ews104 for mac-highsierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-highsierra-wk2 Platform: Mac OS X 10.13.6
EWS Watchlist
Comment 6
2019-06-28 16:51:29 PDT
Comment on
attachment 373151
[details]
Patch
Attachment 373151
[details]
did not pass ios-sim-ews (ios-simulator-wk2): Output:
https://webkit-queues.webkit.org/results/12607109
New failing tests: webrtc/pc-detached-document.html
EWS Watchlist
Comment 7
2019-06-28 16:51:31 PDT
Created
attachment 373165
[details]
Archive of layout-test-results from ews122 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews122 Port: ios-simulator-wk2 Platform: Mac OS X 10.14.5
youenn fablet
Comment 8
2019-07-07 13:54:48 PDT
Created
attachment 373602
[details]
Patch
Devin Rousso
Comment 9
2019-07-07 19:09:47 PDT
Comment on
attachment 373602
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=373602&action=review
The inspector parts of this look fine to me. Is there a test for it (inspector-specific or otherwise)?
> Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp:96 > + if (page && page->settings().webRTCEncryptionEnabled())
Should this be `!page->settings().webRTCEncryptionEnabled()`?
youenn fablet
Comment 10
2019-07-08 10:54:38 PDT
Created
attachment 373642
[details]
Patch for landing
youenn fablet
Comment 11
2019-07-08 10:56:01 PDT
(In reply to Devin Rousso from
comment #9
)
> Comment on
attachment 373602
[details]
> Patch > > View in context: >
https://bugs.webkit.org/attachment.cgi?id=373602&action=review
> > The inspector parts of this look fine to me. Is there a test for it > (inspector-specific or otherwise)?
I added a test for the webrtc switch. I did manual testing for the web inspector part.
> > Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp:96 > > + if (page && page->settings().webRTCEncryptionEnabled()) > > Should this be `!page->settings().webRTCEncryptionEnabled()`?
Fixed.
Devin Rousso
Comment 12
2019-07-15 10:35:14 PDT
Has this landed?
WebKit Commit Bot
Comment 13
2019-07-15 11:59:25 PDT
Comment on
attachment 373642
[details]
Patch for landing Clearing flags on attachment: 373642 Committed
r247438
: <
https://trac.webkit.org/changeset/247438
>
WebKit Commit Bot
Comment 14
2019-07-15 11:59:26 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug