#5928 Fix secure reply-all issues in Gmail context menu

[martgil]

This PR Fixes FlowCrypt secure reply invocation wrongly defaults to "reply" when accessed in Gmail's context menu.

close #5928

---

Tests (delete all except exactly one):

• Tests added or updated

---

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

• addresses the issue it closes (if any)
• code is readable and understandable
• is accompanied with tests, or tests are not needed
• is free of vulnerabilities
• is documented clearly and usefully, or doesn't need documentation

[sosnovsky]

Hi @martgil, in this PR the only thing left is to fix failing tests or some additional code fixes are needed too?

[martgil]

Hello @sosnovsky its all good but the tests keep failing. ill rerun the tests when i got home.

[sosnovsky]

Hi @martgil, tests not pass because of failing Keep original reply message when switching to secure mode live test - maybe some changes affected it?

[martgil]

Sometimes, the test Keep original reply message when switching to secure mode passes when I monitored which tests fails. If it continues to occur, I'll definitely need to fix that.

[martgil]

I can now confirm that Keep original reply message when switching to secure mode is not affected by my changes but is failing too often. I'll check other tests that are often fail and see if there any workaround for them.

[sosnovsky]

It seems Keep original reply message when switching to secure mode fails most often, let's mark it as test.skip for now and fix in another PR, as tested functionality works well, looks like some implementation issue there.

[martgil]

@sosnovsky - The test just started to work fine in the current test. The other test that fails so often is passed test, finalize for mail.google.com - secure reply btn, reply draft but not it passes

I'll let it finish and if not, then I'll create the non-live test PR for it to be merged.

[martgil]

@sosnovsky - would you like to enable auto-merge on this PR?

[sosnovsky]

Yes, it seems to work well and fix all reported issues, can be merged once tests are passing.

[martgil]

Let's move on with the new PR @sosnovsky - tests are failing quite too often? Or maybe, a non-live test PR will do?

[sosnovsky]

Let's move on with the new PR @sosnovsky - tests are failing quite too often?

I think it mostly happens with live tests, as mock tests usually pass without fails

[martgil]

@sosnovsky This one is ready for a review. Thanks!

[sosnovsky]

All good, thanks!