The default bug view has changed. See this FAQ.

Fix breakpoint handling for Fennec and B2G browser actors

RESOLVED FIXED in Firefox 15

Status

()

Firefox
Developer Tools: Debugger
P2
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: past, Assigned: past)

Tracking

Trunk
Firefox 16
Points:
---

Firefox Tracking Flags

(firefox15 fixed)

Details

Attachments

(1 attachment)

Bug 737808 changed the way breakpoints are handled in the debugger server, but only desktop Firefox was updated. Fennec and B2G need the same changes in order for breakpoint setting to work. Bug 751226 will make sure we never get into this situation again, but it may be a few more weeks until I can get to it.
Created attachment 632629 [details] [diff] [review]
Combined patch

Simple fix. B2G needed an additional fix for including the XPCOM helper in the packaging, in order for the debugger to work in a real device (as opposed to desktop b2g).
Attachment #632629 - Flags: review?(mark.finkle)
Attachment #632629 - Flags: review?(21)
Comment on attachment 632629 [details] [diff] [review]
Combined patch

r+ on the fennec parts
Attachment #632629 - Flags: review?(mark.finkle) → review+
Attachment #632629 - Flags: review?(21) → review+
https://hg.mozilla.org/integration/fx-team/rev/864c3502d038
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/864c3502d038
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 16
Whiteboard: [fixed-in-fx-team]
Comment on attachment 632629 [details] [diff] [review]
Combined patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): New feature
User impact if declined: Fennec 15 will have broken breakpoint support in the debugger
Testing completed (on m-c, etc.): On m-c
Risk to taking this patch (and alternatives if risky): this is a pretty straight-forward patch that only adds two new single-line methods. There is the same change for b2g as well which does not impact Fennec in any way, which could be left out if people are worried for some reason.
String or UUID changes made by this patch: none
Attachment #632629 - Flags: approval-mozilla-aurora?

Comment 6

5 years ago
Comment on attachment 632629 [details] [diff] [review]
Combined patch

[Triage Comment]
Fix for a new feature, risk to b2g but we're not shipping b2g off of 15 so go for it.
Attachment #632629 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Whiteboard: [land-in-aurora]
https://hg.mozilla.org/releases/mozilla-aurora/rev/d41b2d8f0630
status-firefox15: --- → fixed
Whiteboard: [land-in-aurora]
You need to log in before you can comment on or make changes to this bug.