Closed
Bug 758799
Opened 13 years ago
Closed 12 years ago
[Mac] remove mIsExpired from mozAccessible
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: hub, Assigned: hub)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
davidb
:
review+
|
Details | Diff | Splinter Review |
remove mExpired from mozAccessible
See https://bugzilla.mozilla.org/show_bug.cgi?id=758304#c5
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → hub
Assignee | ||
Updated•12 years ago
|
Blocks: osxa11y
Summary: [Mac] remove mExpired from mozAccessible → [Mac] remove mIsExpired from mozAccessible
Assignee | ||
Comment 1•12 years ago
|
||
And at the same time check for mGeckoAccessible in the various places to avoid crash.
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Attachment #632388 -
Flags: review?(surkov.alexander)
Assignee | ||
Comment 3•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Attachment #632388 -
Attachment is obsolete: true
Attachment #632388 -
Flags: review?(surkov.alexander)
Assignee | ||
Updated•12 years ago
|
Attachment #632489 -
Flags: review?(surkov.alexander)
Assignee | ||
Updated•12 years ago
|
Attachment #632489 -
Flags: review?(surkov.alexander) → review?(dbolter)
Comment 4•12 years ago
|
||
Comment on attachment 632489 [details] [diff] [review]
Check accessible expiry properly. r=
Review of attachment 632489 [details] [diff] [review]:
-----------------------------------------------------------------
r=me
::: accessible/src/mac/mozAccessible.mm
@@ +561,5 @@
> }
>
> - (BOOL)isEnabled
> {
> + return mGeckoAccessible && ((mGeckoAccessible->InteractiveState() & states::UNAVAILABLE) == 0);
I would habitually enclose this in ()'s but ok. I can't recall our agreed style on this.
Attachment #632489 -
Flags: review?(dbolter) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
You need to log in
before you can comment on or make changes to this bug.
Description
•