Re-enable native stack walking on Android

RESOLVED FIXED in Firefox 54

Status

()

defect
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox53 unaffected, firefox54 fixed, firefox55 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

2 years ago
Bug 1339695 part 8 accidentally disabled native stack walking on Android by
using GP_arm_android instead of GP_PLAT_arm_android in a #if. This patch fixes
that. It also fixes a couple of compile errors that crept into the relevant
code while it was disabled.
Attachment #8850850 - Flags: review?(mstange) → review+

Comment 3

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/096b992d0b34
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
(Assignee)

Comment 4

2 years ago
Comment on attachment 8850850 [details] [diff] [review]
Re-enable native stack walking on Android

Approval Request Comment

[Feature/Bug causing the regression]: Bug 1339695

[User impact if declined]: No native stack walking done by the profiler on Android.

[Is this code covered by automated tests?]: Yes.

[Has the fix been verified in Nightly?]: No.

[Needs manual test from QE? If yes, steps to reproduce]: No.

[List of other uplifts needed for the feature/fix]: None.

[Is the change risky?]: No.

[Why is the change risky/not risky?]: Very simple patch, just re-enables some code that was accidentally disable; and the regression is recent.

[String changes made/needed]: None.
Attachment #8850850 - Flags: approval-mozilla-aurora?
Comment on attachment 8850850 [details] [diff] [review]
Re-enable native stack walking on Android

Re-enable native stack walking on Android. Aurora54+.
Attachment #8850850 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
has problems to apply to aurora:

grafting 407897:096b992d0b34 "Bug 1350211 - Re-enable native stack walking on Android. r=mstange."
merging tools/profiler/core/platform.cpp
warning: conflicts while merging tools/profiler/core/platform.cpp! (edit, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue
(use 'hg resolve' and 'hg graft --continue'
Flags: needinfo?(n.nethercote)
Flags: needinfo?(n.nethercote)
(Assignee)

Comment 10

2 years ago
Comment on attachment 8852308 [details] [diff] [review]
Re-enable native stack walking on Android

This is the m-c patch rebased for aurora.
Flags: needinfo?(n.nethercote)
Attachment #8852308 - Flags: approval-mozilla-aurora?
Attachment #8852308 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.