Closed Bug 1448697 Opened 2 years ago Closed 2 years ago

New code to kill already running Firefox instance doesn't work on my phone (kill: Operation not permitted)

Categories

(Firefox for Android :: Testing, defect)

Firefox 61
Unspecified
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 61
Tracking Status
firefox59 --- unaffected
firefox60 --- unaffected
firefox61 --- fixed

People

(Reporter: JanH, Assigned: gbrown)

References

Details

(Keywords: regression)

Attachments

(2 files)

The implementation provided in bug 1340233 worked flawlessly.

Recently, something must have changed because now I'm getting the attached error message if I try to run a test with my local Firefox build still running on my phone.
Assignee: nobody → gbrown
Sorry :JanH -- I should have remembered this issue.
Blocks: 1440714
This works fine for me locally on the emulator and with my (rooted) device.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=019210527a83b3de487bfb9b9787227cd7ef90ac

:JanH - Does this work for you?
Attachment #8962351 - Flags: review?(bob)
Attachment #8962351 - Flags: feedback?(jh+bugzilla)
Comment on attachment 8962351 [details] [diff] [review]
use stop_application rather than pkill to stop the browser in most cases

Review of attachment 8962351 [details] [diff] [review]:
-----------------------------------------------------------------

r+
Attachment #8962351 - Flags: review?(bob) → review+
Comment on attachment 8962351 [details] [diff] [review]
use stop_application rather than pkill to stop the browser in most cases

Review of attachment 8962351 [details] [diff] [review]:
-----------------------------------------------------------------

Yes, it's working again. Thanks.
Attachment #8962351 - Flags: review?(bob)
Attachment #8962351 - Flags: review+
Attachment #8962351 - Flags: feedback?(jh+bugzilla)
Attachment #8962351 - Flags: feedback+
Attachment #8962351 - Flags: review?(bob) → review+
Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5076991cfc93
Try to use force-stop instead of kill to end remote applications; r=bc
https://hg.mozilla.org/mozilla-central/rev/5076991cfc93
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
You need to log in before you can comment on or make changes to this bug.