Closed Bug 1303133 Opened 4 years ago Closed 4 years ago

Implement ia2AccessibleValue methods in ProxyAccessible for Windows

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: michael.li11702, Assigned: michael.li11702)

References

Details

Attachments

(1 file, 2 obsolete files)

MozReview-Commit-ID: 5nM9Q9Adtir
Attachment #8791750 - Flags: review?(tbsaunde+mozbugs)
Assignee: nobody → mili
Blocks: 1288839
Comment on attachment 8791750 [details] [diff] [review]
Implement ia2AccessibleValue methods in ProxyAccessible for Windows

This needs to be rebased on top of the shared header and QueryInterface wrapper patches, but it seems fine otherwise.
Attachment #8791750 - Flags: review?(tbsaunde+mozbugs)
oh, and you need to check the varient is actually holding a double before getting it out of the varient.
Depends on: 1303040
Attachment #8792125 - Flags: review?(tbsaunde+mozbugs)
Attachment #8791750 - Attachment is obsolete: true
Attachment #8792125 - Attachment is obsolete: true
Attachment #8792125 - Flags: review?(tbsaunde+mozbugs)
Comment on attachment 8801862 [details] [diff] [review]
implement the value related methods in the windows ProxyAccessible r?aklotz

Review of attachment 8801862 [details] [diff] [review]:
-----------------------------------------------------------------

::: accessible/ipc/win/ProxyAccessible.cpp
@@ +335,5 @@
> +  VariantInit(&currentValue);
> +  currentValue.vt = VT_R8;
> +  currentValue.dblVal = aValue;
> +  HRESULT hr = acc->setCurrentValue(currentValue);
> +  return !FAILED(hr);

s/!FAILED/SUCCEEDED/
Attachment #8801862 - Flags: review?(aklotz) → review+
Pushed by tsaunders@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/81a257aa9c4e
Implement ia2AccessibleValue methods in ProxyAccessible for Windows. r=aklotz
https://hg.mozilla.org/mozilla-central/rev/81a257aa9c4e
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.