Open Bug 1416412 Opened 8 years ago Updated 3 years ago

Enclose accessibility service's gConsumers into its own class.

Categories

(Thunderbird :: Disability Access, defect)

defect

Tracking

(Not tracked)

People

(Reporter: yzen, Unassigned)

References

(Blocks 1 open bug)

Details

Should be something like: class Consumers { void Set() void Unset(); void Clear(); bool Has() private: int mConsumers; }
Does this have practical implications, some function is broken?
Flags: needinfo?(yzenevich)
(In reply to Yura Zenevich [:yzen] from comment #0) > Should be something like: > class Consumers { > void Set() > void Unset(); > void Clear(); > bool Has() > private: > int mConsumers; > } I don't believe anything is broken. This was proposed by @surkov primarily for having a centralized place where consumers of a11y services are handled.
Flags: needinfo?(yzenevich)
(In reply to Yura Zenevich [:yzen] from comment #2) > I don't believe anything is broken. This was proposed by @surkov primarily > for having a centralized place where consumers of a11y services are handled. yep, just encapsulation to scope access to the consumers.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.