Implement batch-updating of Touch Bar inputs

RESOLVED FIXED in Firefox 68

Status

()

enhancement
P3
minor
RESOLVED FIXED
4 months ago
3 months ago

People

(Reporter: harry, Assigned: harry)

Tracking

unspecified
mozilla68
Desktop
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

4 months ago

The method that updates the Touch Bar can be called many times per location change. This problem is exacerbated by the work in bug 1522039, which adds additional updates for the Touch Bar's back/forward buttons.

I will implement nsTouchBarUpdater::UpdateTouchBarInputs, which will allow batch-updating of inputs, so there is only one call per location change. This work was suggested by :mikedeboer in the work on bug 1522039.

Assignee

Updated

4 months ago
Depends on: 1522039
Assignee

Comment 1

4 months ago

Changes updateTouchBarInput to a batch-updating updateTouchBarInputs.
Also adds a check for a cached localized title to getTouchBarInput().

Updated

3 months ago
Priority: -- → P3

Comment 2

3 months ago
Pushed by mdeboer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e49c905f4cf6
Implement batch-updating of Touch Bar inputs. r=mikedeboer,spohl

Comment 3

3 months ago
bugherder
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.