Refactor NewWindow by using struct

examples/chat/client/client.go

@@ -17,7 +17,9 @@ func main() {
 	thrust.InitLogger()
 	thrust.Start()
 
-	thrustWindow := thrust.NewWindow(fmt.Sprintf("http://127.0.0.1:%d", *port), nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: fmt.Sprintf("http://127.0.0.1:%d", *port),
+	})
 	thrustWindow.Show()
 	thrustWindow.Focus()
 	// BLOCKING - Dont run before youve excuted all commands you want first.

examples/chat/server/server.go

@@ -145,7 +145,9 @@ func main() {
 
 	thrust.InitLogger()
 	thrust.Start()
-	thrustWindow := thrust.NewWindow(fmt.Sprintf("http://127.0.0.1:%d", *port), nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: fmt.Sprintf("http://127.0.0.1:%d", *port),
+	})
 	thrustWindow.Show()
 	thrustWindow.Focus()
 

examples/jankybrowser/browser.go

@@ -34,7 +34,9 @@ func main() {
 	thrust.InitLogger()
 	thrust.Start()
 
-	thrustWindow := thrust.NewWindow(fmt.Sprintf("http://127.0.0.1:%d", *port), nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: fmt.Sprintf("http://127.0.0.1:%d", *port),
+	})
 	thrustWindow.Show()
 	thrustWindow.Focus()
 

lib/bindings/window/window.go

@@ -29,6 +29,15 @@ type Window struct {
 	SendChannel      *connection.In `json:"-"`
 }
 
+type Options struct {
+	RootUrl  string
+	Size     SizeHW
+	Title    string
+	IconPath string
+	HasFrame bool
+	Session  *session.Session
+}
+
 func checkUrl(s string) (string, error) {
 	u, err := url.Parse(s)
 	if err != nil {
@@ -47,39 +56,40 @@ func checkUrl(s string) (string, error) {
 	return u.String(), err
 }
 
-func NewWindow(s string, sess *session.Session) *Window {
-	u, _ := checkUrl(s)
-
+func NewWindow(options Options) *Window {
 	w := Window{}
-	w.Url = u
-	if len(w.Url) == 0 {
-		w.Url = "http://google.com"
-	}
+	w.setOptions(options)
 	_, sendChannel := connection.GetCommunicationChannels()
 
+	size := options.Size
+	if options.Size == (SizeHW{}) {
+		size = SizeHW{
+			Width:  1024,
+			Height: 768,
+		}
+	}
+
 	windowCreate := Command{
 		Action:     "create",
 		ObjectType: "window",
 		Args: CommandArguments{
-			RootUrl: w.Url,
-			Title:   spawn.ApplicationName,
-			Size: SizeHW{
-				Width:  1024,
-				Height: 768,
-			},
+			RootUrl:  w.Url,
+			Title:    spawn.ApplicationName,
+			Size:     size,
+			HasFrame: !options.HasFrame,
 		},
 	}
 	dispatcher.RegisterHandler(w.DispatchResponse)
-	if sess == nil {
+	if options.Session == nil {
 		w.SetSendChannel(sendChannel)
 		w.WaitingResponses = append(w.WaitingResponses, &windowCreate)
 		w.Send(&windowCreate)
 	} else {
 		go func() {
 			for {
-				if sess.TargetID != 0 {
-					fmt.Println("sess", sess.TargetID)
-					windowCreate.Args.SessionID = sess.TargetID
+				if options.Session.TargetID != 0 {
+					fmt.Println("sess", options.Session.TargetID)
+					windowCreate.Args.SessionID = options.Session.TargetID
 					w.SetSendChannel(sendChannel)
 					w.WaitingResponses = append(w.WaitingResponses, &windowCreate)
 					w.Send(&windowCreate)
@@ -92,6 +102,19 @@ func NewWindow(s string, sess *session.Session) *Window {
 	return &w
 }
 
+func (w *Window) setOptions(options Options) {
+	u, _ := checkUrl(options.RootUrl)
+	w.Url = u
+	if len(w.Url) == 0 {
+		w.Url = "http://google.com"
+	}
+
+	w.Title = options.Title
+	if len(w.Title) == 0 {
+		w.Title = spawn.ApplicationName
+	}
+}
+
 func (w *Window) SetSendChannel(sendChannel *connection.In) {
 	w.SendChannel = sendChannel
 }

lib/commands/commands.go

@@ -46,6 +46,7 @@ type CommandArguments struct {
 	Fullscreen   bool          `json:"fullscreen"`
 	Kiosk        bool          `json:"kiosk"`
 	Focus        bool          `json:"focus"`
+	HasFrame     bool          `json:"has_frame"`
 	Path         string        `json:"path,omitempty"`
 	Message      RemoteMessage `json:"message,omitempty"`
 }

thrust/thrust.go

@@ -21,9 +21,11 @@ Begin Generic Access and Binding Management Section.
 Bindings
 */
 
+type WindowOptions window.Options
+
 /* NewWindow creates a new Window Binding */
-func NewWindow(url string, sess *session.Session) *window.Window {
-	return window.NewWindow(url, sess)
+func NewWindow(options WindowOptions) *window.Window {
+	return window.NewWindow(window.Options(options))
 }
 
 /* NewSession creates a new Session Binding */

tutorials/advanced_session/advanced_session.go

@@ -32,7 +32,10 @@ func main() {
 	   Modified basic_window, where we provide, a session argument
 	   to NewWindow.
 	*/
-	thrustWindow := thrust.NewWindow("http://breach.cc/", mysession)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://breach.cc/",
+		Session: mysession,
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()

tutorials/advanced_window_devtools/advanced_window_devtools.go

@@ -13,7 +13,9 @@ func main() {
 	thrust.SetProvisioner(tutorial.NewTutorialProvisioner())
 	// thrust.Start() must always come before any bindings are created.
 	thrust.Start()
-	thrustWindow := thrust.NewWindow("http://breach.cc/", nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://breach.cc/",
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()

tutorials/basic_menu/basic_menu.go

@@ -11,7 +11,9 @@ func main() {
 	thrust.SetProvisioner(tutorial.NewTutorialProvisioner())
 	// thrust.Start() must always come before any bindings are created.
 	thrust.Start()
-	thrustWindow := thrust.NewWindow("http://breach.cc/", nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://breach.cc/",
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()

tutorials/basic_menu_events/basic_menu_events.go

@@ -15,7 +15,9 @@ func main() {
 	thrust.SetProvisioner(tutorial.NewTutorialProvisioner())
 	// thrust.Start() must always come before any bindings are created.
 	thrust.Start()
-	thrustWindow := thrust.NewWindow("http://breach.cc/", nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://breach.cc/",
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()

tutorials/basic_multiple_windows/basic_multiple_windows.go

@@ -12,12 +12,16 @@ func main() {
 	// thrust.Start() must always come before any bindings are created.
 	thrust.Start()
 
-	thrustWindow := thrust.NewWindow("http://breach.cc/", nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://breach.cc/",
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()
 
-	thrustWindow2 := thrust.NewWindow("http://google.com/", nil)
+	thrustWindow2 := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://google.com/",
+	})
 	thrustWindow2.Show()
 	thrustWindow2.Focus()
 

tutorials/basic_remote_messaging/basic_remote_messaging.go

@@ -22,7 +22,9 @@ func main() {
 	// thrust.Start() must always come before any bindings are created.
 	thrust.Start()
 
-	thrustWindow := thrust.NewWindow("http://localhost:8080/", nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://localhost:8080/",
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()

tutorials/basic_session/basic_session.go

@@ -25,7 +25,10 @@ func main() {
 	  Modified basic_window, where we provide, a session argument
 	  to NewWindow.
 	*/
-	thrustWindow := thrust.NewWindow("http://breach.cc/", mysession)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://breach.cc/",
+		Session: mysession,
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()

tutorials/basic_webserver_app/basic_webserver_app.go

@@ -22,7 +22,10 @@ func main() {
 
 	mysession := thrust.NewSession(false, false, "cache")
 
-	thrustWindow := thrust.NewWindow("http://localhost:8080/", mysession)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://localhost:8080/",
+		Session: mysession,
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()

tutorials/basic_window/basic_window.go

@@ -15,7 +15,9 @@ func main() {
 	// thrust.Start() must always come before any bindings are created.
 	thrust.Start()
 
-	thrustWindow := thrust.NewWindow("http://breach.cc/", nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://breach.cc/",
+	})
 	thrustWindow.Show()
 	thrustWindow.Maximize()
 	thrustWindow.Focus()

tutorials/basic_window_events/basic_window_events.go

@@ -24,7 +24,9 @@ func main() {
 	// thrust.Start() must always come before any bindings are created.
 	thrust.Start()
 
-	thrustWindow := thrust.NewWindow("http://breach.cc/", nil)
+	thrustWindow := thrust.NewWindow(thrust.WindowOptions{
+		RootUrl: "http://breach.cc/",
+	})
 	thrustWindow.Show()
 
 	/*