Minor cleanup

Author: nhooyr

ci/test.mk

@@ -12,6 +12,6 @@ coveralls: gotest
 	goveralls -coverprofile=ci/out/coverage.prof
 
 gotest:
-	go test -covermode=count -coverprofile=ci/out/coverage.prof -coverpkg=./... $${GOTESTFLAGS-} ./...
+	go test -timeout=30m -covermode=count -coverprofile=ci/out/coverage.prof -coverpkg=./... $${GOTESTFLAGS-} ./...
 	sed -i '/stringer\.go/d' ci/out/coverage.prof
 	sed -i '/nhooyr.io\/websocket\/internal\/test/d' ci/out/coverage.prof

conn.go

@@ -1,5 +1,3 @@
-// +build !js
-
 package websocket
 
 // MessageType represents the type of a WebSocket message.

read.go

@@ -120,7 +120,7 @@ func (mr *msgReader) flateContextTakeover() bool {
 }
 
 func (c *Conn) readRSV1Illegal(h header) bool {
-	// If compression is disabled, rsv1 is always illegal.
+	// If compression is disabled, rsv1 is illegal.
 	if !c.flate() {
 		return true
 	}

ws_js.go

@@ -17,18 +17,6 @@ import (
 	"nhooyr.io/websocket/internal/xsync"
 )
 
-// MessageType represents the type of a WebSocket message.
-// See https://tools.ietf.org/html/rfc6455#section-5.6
-type MessageType int
-
-// MessageType constants.
-const (
-	// MessageText is for UTF-8 encoded text messages like JSON.
-	MessageText MessageType = iota + 1
-	// MessageBinary is for binary messages like protobufs.
-	MessageBinary
-)
-
 // Conn provides a wrapper around the browser WebSocket API.
 type Conn struct {
 	ws wsjs.WebSocket