Remove unnecessary call to `_.escape`

What does this MR do?

Fixes a regression where call to _.escape is made second time in updateComment method of notes.js.

Why was this MR needed?

Call to _.escape is already made in getFormData which is used internally by both postComment and updateComment methods, hence additional escaping is not required.

Does this MR meet the acceptance criteria?

• [ ] Changelog entry added, if necessary
• [ ] Documentation created/updated
• [ ] API support added
• Tests
  • [ ] Added for this feature/bug
  • All builds are passing
• Conform by the merge request performance guides
• Conform by the style guides
• Branch has no merge conflicts with master (if it does - rebase it please)
• Squashed related commits together

What are the relevant issue numbers?

Closes #34430 (closed)