After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 616459 - Add needed fields to Orca's keyboardEvent and the ability to validate keyboardEvent Data
Add needed fields to Orca's keyboardEvent and the ability to validate keyboar...
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.30.x
Other All
: Normal normal
: 2.32.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks: 616206 616464
 
 
Reported: 2010-04-22 00:35 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2010-09-20 10:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (6.28 KB, patch)
2010-04-22 00:46 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2010-04-22 00:35:42 UTC
This is a spin-off bug from bug 616206.

While looking at the issues Alejandro was dealing with, and how he was dealing with them, in his work on Orca+Cally, I realized that we had very similar issues with Java (which I want to resolve sooner rather than later).

Therefore, this bug separates out the changes needed in general for Orca's keyboardEvent. Bug 616206 will then focus on the Cally-specific work.
Comment 1 Joanmarie Diggs (IRC: joanie) 2010-04-22 00:46:27 UTC
Created attachment 159295 [details] [review]
proposed patch
Comment 2 Joanmarie Diggs (IRC: joanie) 2010-04-22 00:53:56 UTC
Alex, I took what you had done and then added some changes to see if we can fix the Java issues. Sure enough, it worked. :-)

The attached patch doesn't have the Java fixes; just the common stuff you need for what you're working on in Cally and what I need for Java. I'd appreciate your review.

Thanks!
Comment 3 Alejandro Piñeiro Iglesias (IRC: infapi00) 2010-04-22 11:31:01 UTC
(In reply to comment #2)
> Alex, I took what you had done and then added some changes to see if we can fix
> the Java issues. Sure enough, it worked. :-)
> 
> The attached patch doesn't have the Java fixes; just the common stuff you need
> for what you're working on in Cally and what I need for Java. I'd appreciate
> your review.

After a quick review, it seems ok for me.

As this have the common parts for Cally and Java, I will add a dependency between this bug and the mine, as I would need to update my patch. As far as I see, this bug is already blocking a JAVA one.
Comment 4 Joanmarie Diggs (IRC: joanie) 2010-04-22 13:04:38 UTC
Comment on attachment 159295 [details] [review]
proposed patch

> After a quick review, it seems ok for me.

Okie doke. Thanks!

I also ran Orca's gtk-demo regression tests (the only part of the test suite which is updated for 2.30 *sigh*) and everything passed.

Patch committed to master.