Closed Bug 1518947 Opened 7 months ago Closed 7 months ago

Update breakpad to include the latest Windows/Aarch64 fixes

Categories

(Toolkit :: Crash Reporting, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox66 --- fixed
firefox67 --- fixed

People

(Reporter: gsvelto, Assigned: gsvelto)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1513284 +++

Upstream breakpad has been landing some Windows/AArch64-specific fixes. We should update our code to include version b988fa74ec18de6214b18f723e48331d9a7802ae. Since some of the changes are in the client code this will require applying them manually.

Assignee: nobody → gsvelto
Status: NEW → ASSIGNED
This includes some assorted fixes from upstream plus an adaptation of the
patch in b988fa74ec18de6214b18f723e48331d9a7802ae which includes heap memory
regions in the minidump. Since our support for that is more extensive than
upstream breakpad the resulting change reflects this. Last but not least the
fixes for bug 1489094 and bug 1511140 were split out as patches to be applied
to the unmodified breakpad sources. They will be reintegrated as soon as we
fork breakpad for good.
Attachment #9035951 - Attachment description: Bug 1518947 - Update breakpad to revision b988fa74ec18de6214b18f723e48331d9a7802ae → Bug 1518947 - Update breakpad to revision 44384d80b32a5bb361c2ec3bee667f7ccee566d7
Pushed by gsvelto@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/27334525afa6
Update breakpad to revision 44384d80b32a5bb361c2ec3bee667f7ccee566d7 r=ted
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67

Comment on attachment 9035951 [details]
Bug 1518947 - Update breakpad to revision 44384d80b32a5bb361c2ec3bee667f7ccee566d7

Beta/Release Uplift Approval Request

Feature/Bug causing the regression

New functionality

User impact if declined

Crash reports / crash pings for Windows/AArch64 might not contain all the data we require. This is not visible by the user but will affect developers.

Is this code covered by automated tests?

Yes

Has the fix been verified in Nightly?

Yes

Needs manual test from QE?

No

If yes, steps to reproduce

List of other uplifts needed

None

Risk to taking this patch

Low

Why is the change risky/not risky? (and alternatives if risky)

The change adds a very small amount of code to Gecko to to gather information that is appended to the crash reporter. This just enables some common functionality that's already supported on other architectures and works well. The rest of the changes go into the minidump-analyzer tool which is run outside of Gecko. Both of these bits are already covered by tests.

String changes made/needed

None

Attachment #9035951 - Flags: approval-mozilla-beta?

Comment on attachment 9035951 [details]
Bug 1518947 - Update breakpad to revision 44384d80b32a5bb361c2ec3bee667f7ccee566d7

We want good crash data; let's uplift for beta 6.

Attachment #9035951 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.