Bug 39881 - AX: need to catch NSAccessibilityExceptions in DRT
Summary: AX: need to catch NSAccessibilityExceptions in DRT
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: chris fleizach
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-28 09:33 PDT by chris fleizach
Modified: 2010-05-28 13:58 PDT (History)
4 users (show)

See Also:


Attachments
patch (31.50 KB, patch)
2010-05-28 10:30 PDT, chris fleizach
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description chris fleizach 2010-05-28 09:33:01 PDT
We need to catch NSAccessibilityExceptions in DRT. Normally these are caught in the AX Runtime on the mac, but because DRT is its own AX client, no one is there to catch these otherwise innocuous exceptions.
Comment 1 chris fleizach 2010-05-28 10:30:27 PDT
Created attachment 57341 [details]
patch
Comment 2 Darin Adler 2010-05-28 10:37:37 PDT
Comment on attachment 57341 [details]
patch

Don't we want to give the test access to the exception? Won't we want tests that expect exceptions and other tests that don't expect them? Maybe we can turn the exceptions into JavaScript exceptions?

r=me as is
Comment 3 chris fleizach 2010-05-28 10:41:30 PDT
filed
https://bugs.webkit.org/show_bug.cgi?id=39885

to handle darin's request. 

as of yet, no layout tests need to know about these exceptions, but its possible in the future
Comment 4 chris fleizach 2010-05-28 10:45:39 PDT
http://trac.webkit.org/changeset/60366
Comment 5 chris fleizach 2010-05-28 11:44:34 PDT
apparently tiger doesn't like having a NSMakeRange in a @try block. working on fix now
Comment 6 chris fleizach 2010-05-28 11:46:10 PDT
http://trac.webkit.org/changeset/60372
Comment 8 WebKit Review Bot 2010-05-28 13:58:47 PDT
http://trac.webkit.org/changeset/60372 might have broken GTK Linux 32-bit Debug
The following changes are on the blame list:
http://trac.webkit.org/changeset/60371
http://trac.webkit.org/changeset/60372
http://trac.webkit.org/changeset/60373