compiler/natives/src/time: Don't import strings package.

dmitshur · dmitshur · commit 88d7a4ec38e4 · 2016-06-25T23:45:25.000-07:00
The real time package doesn't import strings package. This keeps the import graph of augmented time package consistent with the real standard library. Source: https://godoc.org/time?import-graph This is a change analogous to #453.
diff --git a/compiler/natives/src/time/time.go b/compiler/natives/src/time/time.go
@@ -4,7 +4,6 @@ package time
 
 import (
 	"runtime"
-	"strings"
 
 	"github.com/gopherjs/gopherjs/js"
 )
@@ -29,8 +28,8 @@ type runtimeTimer struct {
 func initLocal() {
 	d := js.Global.Get("Date").New()
 	s := d.String()
-	i := strings.IndexByte(s, '(')
-	j := strings.IndexByte(s, ')')
+	i := indexByte(s, '(')
+	j := indexByte(s, ')')
 	if i == -1 || j == -1 {
 		localLoc.name = "UTC"
 		return
@@ -95,3 +94,8 @@ func initTestingZone() {
 	z.name = "Local"
 	localLoc = *z
 }
+
+// indexByte is copied from strings package to avoid importing it (since the real time package doesn't).
+func indexByte(s string, c byte) int {
+	return js.InternalObject(s).Call("indexOf", js.Global.Get("String").Call("fromCharCode", c)).Int()
+}
