Run gofmt to add new build constraints

Also remove some 1.13 specific WASM code

api.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

api_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc

api_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

certificate.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

certificate_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

configuration.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

configuration_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc

datachannel.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

datachannel_go_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

datachannel_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc
@@ -162,7 +163,7 @@ func (d *DataChannel) Label() string {
 // out-of-order delivery is allowed.
 func (d *DataChannel) Ordered() bool {
 	ordered := d.underlying.Get("ordered")
-	if jsValueIsUndefined(ordered) {
+	if ordered.IsUndefined() {
 		return true // default is true
 	}
 	return ordered.Bool()
@@ -171,13 +172,13 @@ func (d *DataChannel) Ordered() bool {
 // MaxPacketLifeTime represents the length of the time window (msec) during
 // which transmissions and retransmissions may occur in unreliable mode.
 func (d *DataChannel) MaxPacketLifeTime() *uint16 {
-	if !jsValueIsUndefined(d.underlying.Get("maxPacketLifeTime")) {
+	if !d.underlying.Get("maxPacketLifeTime").IsUndefined() {
 		return valueToUint16Pointer(d.underlying.Get("maxPacketLifeTime"))
-	} else {
-		// See https://bugs.chromium.org/p/chromium/issues/detail?id=696681
-		// Chrome calls this "maxRetransmitTime"
-		return valueToUint16Pointer(d.underlying.Get("maxRetransmitTime"))
 	}
+
+	// See https://bugs.chromium.org/p/chromium/issues/detail?id=696681
+	// Chrome calls this "maxRetransmitTime"
+	return valueToUint16Pointer(d.underlying.Get("maxRetransmitTime"))
 }
 
 // MaxRetransmits represents the maximum number of retransmissions that are

datachannel_js_detach.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc

dtlstransport.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

dtlstransport_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

e2e/e2e_test.go

@@ -1,3 +1,4 @@
+//go:build e2e
 // +build e2e
 
 package main

examples/broadcast/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/custom-logger/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/data-channels-detach/jsfiddle/main.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package main

examples/data-channels/jsfiddle/main.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package main

examples/ice-single-port/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/ice-tcp/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/insertable-streams/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/ortc/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/play-from-disk-renegotation/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/play-from-disk/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/reflect/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/rtcp-processing/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/rtp-forwarder/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/rtp-to-webrtc/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/save-to-disk/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/simulcast/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/swap-tracks/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

examples/vnet/show-network-usage/main.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package main

ice_go.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

icegatherer.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

icegatherer_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

iceserver.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

iceserver_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc

iceserver_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

icetransport.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

icetransport_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

interceptor.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

interceptor_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

js_compare.go

@@ -1,13 +0,0 @@
-// +build js,go1.14
-
-package webrtc
-
-import "syscall/js"
-
-func jsValueIsUndefined(v js.Value) bool {
-	return v.IsUndefined()
-}
-
-func jsValueIsNull(v js.Value) bool {
-	return v.IsNull()
-}

js_compare_legacy.go

@@ -1,13 +0,0 @@
-// +build js,!go1.14
-
-package webrtc
-
-import "syscall/js"
-
-func jsValueIsUndefined(v js.Value) bool {
-	return v == js.Undefined()
-}
-
-func jsValueIsNull(v js.Value) bool {
-	return v == js.Null()
-}

js_utils.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc
@@ -42,7 +43,7 @@ func awaitPromise(promise js.Value) (js.Value, error) {
 }
 
 func valueToUint16Pointer(val js.Value) *uint16 {
-	if jsValueIsNull(val) || jsValueIsUndefined(val) {
+	if val.IsNull() || val.IsUndefined() {
 		return nil
 	}
 	convertedVal := uint16(val.Int())
@@ -50,7 +51,7 @@ func valueToUint16Pointer(val js.Value) *uint16 {
 }
 
 func valueToStringPointer(val js.Value) *string {
-	if jsValueIsNull(val) || jsValueIsUndefined(val) {
+	if val.IsNull() || val.IsUndefined() {
 		return nil
 	}
 	stringVal := val.String()
@@ -79,28 +80,28 @@ func interfaceToValueOrUndefined(val interface{}) js.Value {
 }
 
 func valueToStringOrZero(val js.Value) string {
-	if jsValueIsUndefined(val) || jsValueIsNull(val) {
+	if val.IsUndefined() || val.IsNull() {
 		return ""
 	}
 	return val.String()
 }
 
 func valueToUint8OrZero(val js.Value) uint8 {
-	if jsValueIsUndefined(val) || jsValueIsNull(val) {
+	if val.IsUndefined() || val.IsNull() {
 		return 0
 	}
 	return uint8(val.Int())
 }
 
 func valueToUint16OrZero(val js.Value) uint16 {
-	if jsValueIsNull(val) || jsValueIsUndefined(val) {
+	if val.IsNull() || val.IsUndefined() {
 		return 0
 	}
 	return uint16(val.Int())
 }
 
 func valueToUint32OrZero(val js.Value) uint32 {
-	if jsValueIsNull(val) || jsValueIsUndefined(val) {
+	if val.IsNull() || val.IsUndefined() {
 		return 0
 	}
 	return uint32(val.Int())

mediaengine.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

mediaengine_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

ortc_datachannel_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

ortc_media_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

ortc_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

peerconnection.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

peerconnection_close_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

peerconnection_go_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

peerconnection_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 // Package webrtc implements the WebRTC 1.0 as defined in W3C WebRTC specification document.
@@ -55,6 +56,7 @@ func (api *API) NewPeerConnection(configuration Configuration) (_ *PeerConnectio
 	}, nil
 }
 
+// JSValue returns the underlying PeerConnection
 func (pc *PeerConnection) JSValue() js.Value {
 	return pc.underlying
 }
@@ -561,7 +563,7 @@ func iceServerToValue(server ICEServer) js.Value {
 }
 
 func valueToConfiguration(configValue js.Value) Configuration {
-	if jsValueIsNull(configValue) || jsValueIsUndefined(configValue) {
+	if configValue.IsNull() || configValue.IsUndefined() {
 		return Configuration{}
 	}
 	return Configuration{
@@ -578,7 +580,7 @@ func valueToConfiguration(configValue js.Value) Configuration {
 }
 
 func valueToICEServers(iceServersValue js.Value) []ICEServer {
-	if jsValueIsNull(iceServersValue) || jsValueIsUndefined(iceServersValue) {
+	if iceServersValue.IsNull() || iceServersValue.IsUndefined() {
 		return nil
 	}
 	iceServers := make([]ICEServer, iceServersValue.Length())
@@ -599,10 +601,10 @@ func valueToICEServer(iceServerValue js.Value) ICEServer {
 }
 
 func valueToICECandidate(val js.Value) *ICECandidate {
-	if jsValueIsNull(val) || jsValueIsUndefined(val) {
+	if val.IsNull() || val.IsUndefined() {
 		return nil
 	}
-	if jsValueIsUndefined(val.Get("protocol")) && !jsValueIsUndefined(val.Get("candidate")) {
+	if val.Get("protocol").IsUndefined() && !val.Get("candidate").IsUndefined() {
 		// Missing some fields, assume it's Firefox and parse SDP candidate.
 		c, err := ice.UnmarshalCandidate(val.Get("candidate").String())
 		if err != nil {
@@ -653,7 +655,7 @@ func sessionDescriptionToValue(desc *SessionDescription) js.Value {
 }
 
 func valueToSessionDescription(descValue js.Value) *SessionDescription {
-	if jsValueIsNull(descValue) || jsValueIsUndefined(descValue) {
+	if descValue.IsNull() || descValue.IsUndefined() {
 		return nil
 	}
 	return &SessionDescription{

peerconnection_media_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

peerconnection_renegotiation_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

rtpreceiver_go.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

rtpreceiver_go_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

rtpreceiver_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc

rtpreceiver_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

rtpsender_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc

rtptransceiver.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

rtptransceiver_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc

rtptransceiver_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

sctptransport.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

sctptransport_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

sdp_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

sdpsemantics_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

settingengine.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

settingengine_js.go

@@ -1,3 +1,4 @@
+//go:build js && wasm
 // +build js,wasm
 
 package webrtc

settingengine_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

srtp_writer_future.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

stats_go.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

stats_go_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

track_local_static_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

track_remote.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

track_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc

vnet_test.go

@@ -1,3 +1,4 @@
+//go:build !js
 // +build !js
 
 package webrtc