1506041 - Code cleanup: Convert more static functions to lambdas in the markup map

Status: RESOLVED FIXED

(Core :: Disability Access APIs, enhancement, P2)

Description

[Marco Zehe (:MarcoZ)]

Right now, we have a wild mix of lambdas in accessible/base/markupmap.h, and a set of static functions in accessible/base/nsAccessibilityService.cpp.

To make this more maintainable for future additions, I am going to convert those static functions that make sense to lambdas in the markup map.

Comment 1

[Marco Zehe (:MarcoZ)]

Attachment: Bug 1506041 - Code cleanup: Convert more static functions to lambdas in the markup map, r=jamie

All static functions from nsAccessibilityService that were either only called once from the markup map, or are simple one-liners that were only called up to three times, were converted to lambdas in the markup map.

the static function that creates HyperTextAccessibleWrap has remained untouched because it is called a lot from the markup map, even though its implementation is actually just a one-liner.

The other untouched static function is the templated one for html:dt or html:dd.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d4a258084934
Code cleanup: Convert more static functions to lambdas in the markup map, r=Jamie

Comment 3

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/d4a258084934