874483 - get rid of some static constructors in accessible/

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Comment 1

[Trevor Saunders (:tbsaunde)]

Attachment: bug 874483 - get rid of some static constructors in accessible/

Comment 2

[alexander :surkov (:asurkov)]

Comment on attachment 752229 [details] [diff] [review]
bug 874483 - get rid of some static constructors in accessible/

Review of attachment 752229 [details] [diff] [review]:
-----------------------------------------------------------------

::: accessible/src/base/nsTextEquivUtils.cpp
@@ +22,5 @@
> +  /**
> +   * The accessible for which we are computing a text equivalent. It is useful
> +   * for bailing out during recursive text computation, or for special cases
> +   * like step f. of the ARIA implementation guide.
> +   */

wrong indentation

btw, there's no f step any more but it seems we can nothing to do with it now

Comment 3

[Trevor Saunders (:tbsaunde)]

> btw, there's no f step any more but it seems we can nothing to do with it now

are you saying we should completely rm it?

Comment 4

[alexander :surkov (:asurkov)]

no, no, just leave it as is for now, in this case obsolete comment is better than nothing I think.

Comment 5

[Trevor Saunders (:tbsaunde)]

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/7799b182a1a0

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7799b182a1a0