Merge branch 'v10' into node-manager

pkg/app/app.go

@@ -196,26 +196,20 @@ func internalURLChecker() func(string) bool {
 }
 
 func newClientBody(d Dispatcher) *htmlBody {
-	ctx, cancel := context.WithCancel(context.Background())
 	body := &htmlBody{
 		htmlElement: htmlElement{
-			tag:           "body",
-			context:       ctx,
-			contextCancel: cancel,
-			dispatcher:    d,
-			jsElement:     Window().Get("document").Get("body"),
+			tag:        "body",
+			dispatcher: d,
+			jsElement:  Window().Get("document").Get("body"),
 		},
 	}
 	body.setSelf(body)
 
-	ctx, cancel = context.WithCancel(context.Background())
 	content := &htmlDiv{
 		htmlElement: htmlElement{
-			tag:           "div",
-			context:       ctx,
-			contextCancel: cancel,
-			dispatcher:    d,
-			jsElement:     body.JSValue().firstElementChild(),
+			tag:        "div",
+			dispatcher: d,
+			jsElement:  body.JSValue().firstElementChild(),
 		},
 	}
 	content.setSelf(content)

pkg/app/component.go

@@ -1,7 +1,6 @@
 package app
 
 import (
-	"context"
 	"io"
 	"reflect"
 	"strings"
@@ -148,8 +147,6 @@ type resize struct{}
 // Compo represents the base struct to use in order to build a component.
 type Compo struct {
 	disp       Dispatcher
-	ctx        context.Context
-	ctxCancel  func()
 	parentElem UI
 	root       UI
 	this       Composer
@@ -163,8 +160,6 @@ func (c *Compo) JSValue() Value {
 // Mounted reports whether the component is mounted.
 func (c *Compo) Mounted() bool {
 	return c.getDispatcher() != nil &&
-		c.ctx != nil &&
-		c.ctx.Err() == nil &&
 		c.root != nil && c.root.Mounted() &&
 		c.self() != nil
 }
@@ -240,10 +235,6 @@ func (c *Compo) setSelf(v UI) {
 	c.this = nil
 }
 
-func (c *Compo) getContext() context.Context {
-	return c.ctx
-}
-
 func (c *Compo) getDispatcher() Dispatcher {
 	return c.disp
 }
@@ -280,7 +271,6 @@ func (c *Compo) mount(d Dispatcher) error {
 	}
 
 	c.disp = d
-	c.ctx, c.ctxCancel = context.WithCancel(context.Background())
 
 	root := c.render()
 	if err := mount(d, root); err != nil {
@@ -306,7 +296,6 @@ func (c *Compo) mount(d Dispatcher) error {
 }
 
 func (c *Compo) dismount() {
-	c.ctxCancel()
 	dismount(c.root)
 
 	if dismounter, ok := c.this.(Dismounter); ok {

pkg/app/condition.go

@@ -1,7 +1,6 @@
 package app
 
 import (
-	"context"
 	"io"
 
 	"github.com/maxence-charriere/go-app/v9/pkg/errors"
@@ -106,10 +105,6 @@ func (c condition) self() UI {
 func (c condition) setSelf(UI) {
 }
 
-func (c condition) getContext() context.Context {
-	return nil
-}
-
 func (c condition) getDispatcher() Dispatcher {
 	return nil
 }

pkg/app/context.go

@@ -380,7 +380,7 @@ func (ctx uiContext) cryptoKey() string {
 
 func makeContext(src UI) Context {
 	return uiContext{
-		Context:            src.getContext(),
+		Context:            context.Background(),
 		src:                src,
 		jsSrc:              src.JSValue(),
 		appUpdateAvailable: appUpdateAvailable,

pkg/app/html.go

@@ -1,7 +1,6 @@
 package app
 
 import (
-	"context"
 	"io"
 	"reflect"
 	"strconv"
@@ -31,11 +30,9 @@ type htmlElement struct {
 	parent        UI
 	children      []UI
 
-	context       context.Context
-	contextCancel func()
-	dispatcher    Dispatcher
-	jsElement     Value
-	this          UI
+	dispatcher Dispatcher
+	jsElement  Value
+	this       UI
 }
 
 func (e *htmlElement) Tag() string {
@@ -51,7 +48,7 @@ func (e *htmlElement) JSValue() Value {
 }
 
 func (e *htmlElement) Mounted() bool {
-	return e.context != nil && e.context.Err() == nil
+	return e.jsElement != nil
 }
 
 func (e *htmlElement) name() string {
@@ -66,10 +63,6 @@ func (e *htmlElement) setSelf(v UI) {
 	e.this = v
 }
 
-func (e *htmlElement) getContext() context.Context {
-	return e.context
-}
-
 func (e *htmlElement) getDispatcher() Dispatcher {
 	return e.dispatcher
 }
@@ -99,7 +92,6 @@ func (e *htmlElement) mount(d Dispatcher) error {
 		return errors.New("html element is already mounted").WithTag("tag", e.tag)
 	}
 
-	e.context, e.contextCancel = context.WithCancel(context.Background())
 	e.dispatcher = d
 
 	jsElement, err := Window().createElement(e.tag, e.xmlns)
@@ -138,7 +130,7 @@ func (e *htmlElement) dismount() {
 		eh.Dismount()
 	}
 
-	e.contextCancel()
+	e.jsElement = nil
 }
 
 func (e *htmlElement) canUpdateWith(v UI) bool {

pkg/app/node.go

@@ -1,7 +1,6 @@
 package app
 
 import (
-	"context"
 	"io"
 	"reflect"
 	"strings"
@@ -21,7 +20,6 @@ type UI interface {
 	name() string
 	self() UI
 	setSelf(UI)
-	getContext() context.Context
 	getDispatcher() Dispatcher
 	getAttributes() attributes
 	getEventHandlers() eventHandlers

pkg/app/node_test.go

@@ -144,7 +144,6 @@ func testMounted(t *testing.T, n UI) {
 	case *text, *raw:
 
 	default:
-		require.NoError(t, n.getContext().Err())
 		require.NotNil(t, n.self())
 	}
 
@@ -162,7 +161,6 @@ func testDismounted(t *testing.T, n UI) {
 	case *text, *raw:
 
 	default:
-		require.Error(t, n.getContext().Err())
 		require.Nil(t, n.self())
 	}
 

pkg/app/range.go

@@ -1,7 +1,6 @@
 package app
 
 import (
-	"context"
 	"io"
 	"reflect"
 	"sort"
@@ -105,10 +104,6 @@ func (r rangeLoop) self() UI {
 func (r rangeLoop) setSelf(UI) {
 }
 
-func (r rangeLoop) getContext() context.Context {
-	return nil
-}
-
 func (r rangeLoop) getDispatcher() Dispatcher {
 	return nil
 }

pkg/app/raw.go

@@ -1,7 +1,6 @@
 package app
 
 import (
-	"context"
 	"io"
 	"strings"
 
@@ -54,10 +53,6 @@ func (r *raw) self() UI {
 func (r *raw) setSelf(UI) {
 }
 
-func (r *raw) getContext() context.Context {
-	return nil
-}
-
 func (r *raw) getDispatcher() Dispatcher {
 	return r.disp
 }

pkg/app/text.go

@@ -1,7 +1,6 @@
 package app
 
 import (
-	"context"
 	"fmt"
 	"html"
 	"io"
@@ -45,10 +44,6 @@ func (t *text) self() UI {
 func (t *text) setSelf(n UI) {
 }
 
-func (t *text) getContext() context.Context {
-	return context.TODO()
-}
-
 func (t *text) getDispatcher() Dispatcher {
 	return t.disp
 }