Closed
Bug 1192330
Opened 9 years ago
Closed 9 years ago
prepare text change data to work with proxied text change events
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: tbsaunde, Assigned: tbsaunde)
References
Details
Attachments
(4 files)
5.73 KB,
patch
|
surkov
:
review+
|
Details | Diff | Splinter Review |
6.32 KB,
patch
|
surkov
:
review+
|
Details | Diff | Splinter Review |
3.79 KB,
patch
|
surkov
:
review+
|
Details | Diff | Splinter Review |
1010 bytes,
patch
|
surkov
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•9 years ago
|
||
This will enable us to remove the virtual function GetModifiedText().
Attachment #8645052 -
Flags: review?(surkov.alexander)
Assignee | ||
Comment 2•9 years ago
|
||
There's no point in this indirection since ia2AccessibleText has direct access to this data.
Attachment #8645053 -
Flags: review?(surkov.alexander)
Assignee | ||
Comment 3•9 years ago
|
||
Soon we will need to be able to update the text change event data from both HandleAccEvent() and ProxyTextChangeEvent(), so separate out the logic to do that into a function.
Attachment #8645054 -
Flags: review?(surkov.alexander)
Updated•9 years ago
|
Attachment #8645052 -
Flags: review?(surkov.alexander) → review+
Updated•9 years ago
|
Attachment #8645053 -
Flags: review?(surkov.alexander) → review+
Comment 4•9 years ago
|
||
Comment on attachment 8645054 [details] [diff] [review] add ia2AccessibleText::UpdateTextChangeData Review of attachment 8645054 [details] [diff] [review]: ----------------------------------------------------------------- ::: accessible/windows/msaa/HyperTextAccessibleWrap.cpp @@ +58,5 @@ > + static_cast<HyperTextAccessibleWrap*>(accessible->AsHyperText()); > + ia2AccessibleText::UpdateTextChangeData(text, event->IsTextInserted(), > + event->ModifiedText(), > + event->GetStartOffset(), > + event->GetLength()); it might nicer if UpdateTextChangeData was taking AccTextChangeEvent as argument, at least less variables on stack.
Attachment #8645054 -
Flags: review?(surkov.alexander) → review+
Assignee | ||
Comment 5•9 years ago
|
||
(In reply to alexander :surkov from comment #4) > Comment on attachment 8645054 [details] [diff] [review] > add ia2AccessibleText::UpdateTextChangeData > > Review of attachment 8645054 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: accessible/windows/msaa/HyperTextAccessibleWrap.cpp > @@ +58,5 @@ > > + static_cast<HyperTextAccessibleWrap*>(accessible->AsHyperText()); > > + ia2AccessibleText::UpdateTextChangeData(text, event->IsTextInserted(), > > + event->ModifiedText(), > > + event->GetStartOffset(), > > + event->GetLength()); > > it might nicer if UpdateTextChangeData was taking AccTextChangeEvent as > argument, at least less variables on stack. but ProxyTextChangeEvent() doesn't have one so it wouldn't work there.
Assignee | ||
Comment 6•9 years ago
|
||
Attachment #8645126 -
Flags: review?(lorien)
Assignee | ||
Updated•9 years ago
|
Attachment #8645126 -
Flags: review?(lorien) → review?(surkov.alexander)
Updated•9 years ago
|
Attachment #8645126 -
Flags: review?(surkov.alexander) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/5c1af6e4c004 https://hg.mozilla.org/integration/mozilla-inbound/rev/d5b9c388f05a https://hg.mozilla.org/integration/mozilla-inbound/rev/1409d0fcea5d https://hg.mozilla.org/integration/mozilla-inbound/rev/2f76881caf22
https://hg.mozilla.org/mozilla-central/rev/5c1af6e4c004 https://hg.mozilla.org/mozilla-central/rev/d5b9c388f05a https://hg.mozilla.org/mozilla-central/rev/1409d0fcea5d https://hg.mozilla.org/mozilla-central/rev/2f76881caf22
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Updated•7 years ago
|
Assignee: nobody → tbsaunde+mozbugs
You need to log in
before you can comment on or make changes to this bug.
Description
•