1680167 - Mark nsFocusManager::SetFocus as MOZ_CAN_RUN_SCRIPT

Status: RESOLVED FIXED

(Core :: DOM: UI Events & Focus Handling, task, P3)

Description

[Kagami Rosylight [:saschanaz] (they/them)]

Because it can fire a synchronous focus event.

Comment 1

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 1: MOZ_CAN_RUN_SCRIPT_BOUNDARY in accessible/ r=masayuki

Comment 2

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 2: MOZ_CAN_RUN_SCRIPT in dom/html r=masayuki

Depends on D98624

Comment 3

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 3: MOZ_CAN_RUN_SCRIPT_BOUNDARY in dom/base r=masayuki

Depends on D98625

Comment 4

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 4: MOZ_CAN_RUN_SCRIPT in dom/xul r=masayuki

Depends on D98626

Comment 5

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 5: MOZ_CAN_RUN_SCRIPT_BOUNDARY in dom/* r=masayuki

Depends on D98627

Comment 6

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 6: Use RefPtr for nsFocusManager in editor r=masayuki

Depends on D98628

Comment 7

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 7: MOZ_CAN_RUN_SCRIPT_BOUNDARY in layout r=masayuki

Depends on D98629

Comment 8

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 8: MOZ_CAN_RUN_SCRIPT_BOUNDARY in toolkit r=masayuki

Depends on D98630

Comment 9

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1680167 - Part 9: Mark setFocus() as [can_run_script] r=masayuki

Depends on D98631

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2674197fb4e7
Part 1: MOZ_CAN_RUN_SCRIPT_BOUNDARY in accessible/ r=masayuki
https://hg.mozilla.org/integration/autoland/rev/94342ecd2ca3
Part 2: MOZ_CAN_RUN_SCRIPT in dom/html r=masayuki
https://hg.mozilla.org/integration/autoland/rev/7a8a4aad88d4
Part 3: MOZ_CAN_RUN_SCRIPT_BOUNDARY in dom/base r=masayuki
https://hg.mozilla.org/integration/autoland/rev/d6580f14dc79
Part 4: MOZ_CAN_RUN_SCRIPT in dom/xul r=masayuki
https://hg.mozilla.org/integration/autoland/rev/95de969a1cef
Part 5: MOZ_CAN_RUN_SCRIPT_BOUNDARY in dom/* r=masayuki
https://hg.mozilla.org/integration/autoland/rev/1a52669779d5
Part 6: Use RefPtr for nsFocusManager in editor r=masayuki
https://hg.mozilla.org/integration/autoland/rev/e8be9b56e318
Part 7: MOZ_CAN_RUN_SCRIPT_BOUNDARY in layout r=masayuki
https://hg.mozilla.org/integration/autoland/rev/fa5112de4b90
Part 8: MOZ_CAN_RUN_SCRIPT_BOUNDARY in toolkit r=masayuki
https://hg.mozilla.org/integration/autoland/rev/298d9590e8c9
Part 9: Mark setFocus() as [can_run_script] r=masayuki

Comment 11

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/2674197fb4e7
https://hg.mozilla.org/mozilla-central/rev/94342ecd2ca3
https://hg.mozilla.org/mozilla-central/rev/7a8a4aad88d4
https://hg.mozilla.org/mozilla-central/rev/d6580f14dc79
https://hg.mozilla.org/mozilla-central/rev/95de969a1cef
https://hg.mozilla.org/mozilla-central/rev/1a52669779d5
https://hg.mozilla.org/mozilla-central/rev/e8be9b56e318
https://hg.mozilla.org/mozilla-central/rev/fa5112de4b90
https://hg.mozilla.org/mozilla-central/rev/298d9590e8c9