app: don't schedule a new frame for profiling events

Sometimes it's useful to profile yet not continously re-draw. If the programs
wants the old behaviour, it can issue an InvalidateOp or call
Window.Invalidate.

Signed-off-by: Elias Naur <mail@eliasnaur.com>

app/internal/input/router.go

@@ -30,7 +30,9 @@ type Router struct {
 	wakeupTime time.Time
 
 	// ProfileOp summary.
-	profHandlers []event.Key
+	profiling    bool
+	profHandlers map[event.Key]struct{}
+	profile      profile.Event
 }
 
 type handlerEvents struct {
@@ -39,13 +41,21 @@ type handlerEvents struct {
 }
 
 func (q *Router) Events(k event.Key) []event.Event {
-	return q.handlers.Events(k)
+	events := q.handlers.Events(k)
+	if _, isprof := q.profHandlers[k]; isprof {
+		delete(q.profHandlers, k)
+		events = append(events, q.profile)
+	}
+	return events
 }
 
 func (q *Router) Frame(ops *op.Ops) {
 	q.handlers.Clear()
 	q.wakeup = false
-	q.profHandlers = q.profHandlers[:0]
+	q.profiling = false
+	for k := range q.profHandlers {
+		delete(q.profHandlers, k)
+	}
 	q.reader.Reset(ops)
 	q.collect()
 
@@ -82,19 +92,21 @@ func (q *Router) collect() {
 			}
 		case opconst.TypeProfile:
 			op := decodeProfileOp(encOp.Data, encOp.Refs)
-			q.profHandlers = append(q.profHandlers, op.Key)
+			if q.profHandlers == nil {
+				q.profHandlers = make(map[event.Key]struct{})
+			}
+			q.profiling = true
+			q.profHandlers[op.Key] = struct{}{}
 		}
 	}
 }
 
-func (q *Router) AddProfile(e profile.Event) {
+func (q *Router) AddProfile(profile profile.Event) {
-	for _, h := range q.profHandlers {
+	q.profile = profile
-		q.handlers.Add(h, e)
-	}
 }
 
 func (q *Router) Profiling() bool {
-	return len(q.profHandlers) > 0
+	return q.profiling
 }
 
 func (q *Router) WakeupTime() (time.Time, bool) {

app/window.go

@@ -145,7 +145,6 @@ func (w *Window) draw(size image.Point, frame *op.Ops) {
 		q := 100 * time.Microsecond
 		timings := fmt.Sprintf("tot:%7s cpu:%7s %s", frameDur.Round(q), drawDur.Round(q), w.gpu.Timings())
 		w.queue.q.AddProfile(profile.Event{Timings: timings})
-		w.setNextFrame(time.Time{})
 	}
 	if t, ok := w.queue.q.WakeupTime(); ok {
 		w.setNextFrame(t)