Closed Bug 1147079 Opened 5 years ago Closed 5 years ago

IPC Proxy for Table

Categories

(Core :: Disability Access APIs, defect)

36 Branch
x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: smaug, Assigned: smaug)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

The patch is getting large enough that atk part should land in a separate patch or bug.
Blocks: 1147080
Attached patch v1Splinter Review
This has now systematically all of TableAccessible converted (excluding AsAccessible() method).
Whether or not all the methods are needed becomes clearer once the backends are
converted to use the proxy.

Decided to use Table* as a prefix for the method to make ProxyAccessible a bit
easier to understand.
Attachment #8582592 - Flags: review?(tbsaunde+mozbugs)
Comment on attachment 8582592 [details] [diff] [review]
v1


>+  void TableColumnDescription(uint32_t& aCol, nsString& aDescription);

just uint32_t seems much better
Attachment #8582592 - Flags: review?(tbsaunde+mozbugs) → review+
Attached patch v2Splinter Review
https://hg.mozilla.org/mozilla-central/rev/d92c07d1e6df
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.