1881078 - Add Services.intl.stringHasRTLChars to expose StringHasRTLChars in JS

Status: RESOLVED FIXED

(Core :: Internationalization, task)

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1881078 - Add Services.intl.stringHasRTLChars . r=jfkthame.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c4dcd746b1b5
Add Services.intl.stringHasRTLChars . r=jfkthame.

Comment 3

[Sandor Molnar[:smolnar]]

Backed out for causing dt failures @ devtools/client/jsonview/test/browser_jsonview_copy_json.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/29502e93e5a1dafb93c2d85c64676fe8b5ce1cb8

Push with failures

Failure log -> TEST-UNEXPECTED-FAIL | devtools/client/jsonview/test/browser_jsonview_copy_json.js | There must be proper JSON displayed - Got "nameInvalid object", expected "name"value""

Comment 4

[Nicolas Chevobbe [:nchevobbe]]

ah yes, I forgot I need to handle the JSONViewer, let me do that

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b12123324ea7
Add Services.intl.stringHasRTLChars . r=jfkthame.

Comment 6

[Cristina Horotan [:chorotan]]

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

Comment 7

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1881078 - Add Services.intl.stringHasRTLChars . r=jfkthame.

Original Revision: https://phabricator.services.mozilla.com/D202241

Comment 8

[Phabricator Automation]

Uplift Approval Request

• String changes made/needed: -
• Needs manual QE test: no
• Risk associated with taking this patch: low
• Explanation of risk level: Existing function exposed in Javascript, covered with test
• Is Android affected?: no
• Code covered by automated testing: yes
• Fix verified in Nightly: no
• User impact if declined: DevTools Console and Debugger will be slower in 124 than it was in 123
• Steps to reproduce for manual QE testing: -

Comment 9

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/1bbb1b6e671a