accessibilityIsIgnoredBase() needs to respect when platform says include

• Component: Accessibility
• Status: RESOLVED FIXED

[chris fleizach]

accessibilityIsIgnoredBase() is not doing this, so when an object is always included by a platform, its not working

[chris fleizach]

accessibilityIsIgnoredBased() needs to return a ternary value

[chris fleizach]

Created attachment 50532 [details]
Patch

[WebKit Review Bot]

Attachment 50532 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/618023

[chris fleizach]

Created attachment 50533 [details]
Patch

[Beth Dakin]

Comment on attachment 50533 [details]
Patch

> -#if PLATFORM(GTK)
> -    return true;
> -#endif

Are you sure you want to remove this?

r=me

[chris fleizach]

(In reply to comment #5)
> (From update of attachment 50533 [details])
> > -#if PLATFORM(GTK)
> > -    return true;
> > -#endif
> 
> Are you sure you want to remove this?
> 
 Yea, i looked at what was going on. GTK wants the platform to decide the fate of that object. Now that that object calls accessibilityIsIgnoredBase(), that's now taken care of

> r=me

[chris fleizach]

http://trac.webkit.org/changeset/55855

[Gustavo Noronha (kov)]

Thanks for the quick fix! Much appreciated =).