Bug 147737 - AX: AXLoadComplete that comes before AX API access won't fire
Summary: AX: AXLoadComplete that comes before AX API access won't fire
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-06 11:54 PDT by Doug Russell
Modified: 2015-08-06 19:47 PDT (History)
11 users (show)

See Also:

Attachments
Patch (4.21 KB, patch)
2015-08-06 12:04 PDT, Doug Russell 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Doug Russell 2015-08-06 11:54:56 PDT 
If a notification, specifically AXLoadComplete, should fire, but no accessibility APIs have been accessed yet, accesssibilityEnabled() will be false and the notification won't fire.
Comment 1 Doug Russell 2015-08-06 12:04:55 PDT 
Created attachment 258383 [details]
Patch
Comment 2 WebKit Commit Bot 2015-08-06 13:40:50 PDT 
Comment on attachment 258383 [details]
Patch

Clearing flags on attachment: 258383

Committed r188055: <http://trac.webkit.org/changeset/188055>
Comment 3 WebKit Commit Bot 2015-08-06 13:40:55 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 4 Simon Fraser (smfr) 2015-08-06 16:48:28 PDT 
This patch broke layout tests:
https://build.webkit.org/results/Apple%20Mavericks%20Release%20WK1%20(Tests)/r188057%20(16777)/results.html

Please fix.
Comment 5 Doug Russell 2015-08-06 16:59:46 PDT 
(In reply to comment #4)
> This patch broke layout tests:
> https://build.webkit.org/results/Apple%20Mavericks%20Release%20WK1%20(Tests)/
> r188057%20(16777)/results.html
> 
> Please fix.

investigating now
Comment 6 Doug Russell 2015-08-06 19:47:49 PDT 
Prepping a patch. Tracking it with https://bugs.webkit.org/show_bug.cgi?id=147763.