Bug 1059602 took care of the misused MFBT_API in security/sandbox/linux/SandboxInternal.h (introduced in bug 1041886), but there are more in Sandbox.h. As long at at least one symbol exported from libmozsandbox and used in libxul exists then bug 1059602 shouldn't reoccur, but that's no excuse not to clean it up now.
Attachment #8482108 - Flags: review?(mh+mozilla) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Comment on attachment 8482108 [details] [diff] [review] bug1061085-more-moz-export-hg0.diff Approval Request Comment [Feature/regressing bug #]: Bug 1041886 [User impact if declined]: Possibly nothing, but the code currently on the 34 branch is incorrect and could cause problems later if future changes are uplifted. [Describe test coverage new/current, TBPL]: Covered by build testing. [Risks and why]: None; if it passes try then it works. [String/UUID change made/needed]: None.
Attachment #8482108 - Flags: approval-mozilla-aurora?
Attachment #8482108 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.