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)
Thunderbird
Disability Access
Tracking
(Not tracked)
NEW
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;
}
Comment 1•8 years ago
|
||
Does this have practical implications, some function is broken?
Flags: needinfo?(yzenevich)
Reporter | ||
Comment 2•8 years ago
|
||
(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)
Comment 3•8 years ago
|
||
(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.
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•