New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update interfaces/webrtc-identity.idl #20944
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@stephenmcgruer can you find the spec change and report the duplication problem here? If a webrtc-pc update is in flight we might have to update both together here :/
@@ -62,6 +66,29 @@ interface RTCIdentityAssertion { | |||
attribute DOMString name; | |||
}; | |||
|
|||
partial interface RTCError { | |||
readonly attribute long? httpRequestStatusCode; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The "Partial interface RTCError: member names are unique" subtest fails, and indeed this is already in https://w3c.github.io/webrtc-pc/#rtcerror-interface.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like w3c/webrtc-pc#2390 missed parts of httpRequestStatusCode
in the move. I'll file a PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was w3c/webrtc-pc#2428. Thanks!
The RTCError duplication problem should be fixed now, PTAL @foolip |
Manual version of #20662 with WebIDL dependency added