The default bug view has changed. See this FAQ.

After reloading a page, the selected script and line should remain the same

RESOLVED FIXED in Firefox 17

Status

()

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

People

(Reporter: vporof, Assigned: vporof)

Tracking

12 Branch
Firefox 17
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixed-in-fx-team])

Attachments

(3 attachments)

(Assignee)

Description

5 years ago
Open debugger, switch to another script and reload the page. Most likely, the currently visible script is switched to the first one in the list. This annoyed me today and I think we should fix it.
(Assignee)

Comment 1

5 years ago
Created attachment 648284 [details] [diff] [review]
v1
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Attachment #648284 - Flags: review?(rcampbell)
(Assignee)

Updated

5 years ago
Priority: -- → P3
Comment on attachment 648284 [details] [diff] [review]
v1

nice little usability win there. With a test!
Attachment #648284 - Flags: review?(rcampbell) → review+
(Assignee)

Comment 3

5 years ago
Created attachment 648355 [details] [diff] [review]
v1.1

Orange fix.
https://tbpl.mozilla.org/?tree=Try&rev=763ff56bc1f3
(Assignee)

Updated

5 years ago
Whiteboard: [land-in-fx-team]
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/integration/fx-team/rev/64bc6b0e9127
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/64bc6b0e9127
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
Depends on: 780198
Depends on: 782179
Backed out for causing the frequent bug 780198 and bug 782179:
https://hg.mozilla.org/mozilla-central/rev/3b46b03dff5c
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [fixed-in-fx-team]
Target Milestone: Firefox 17 → ---
The backout had failures, so I have reverted it (effectively relanding this bug) and instead disabled the test (rather than trying to figure out how to backout cleanly). Sorry for the extra noise.

Relanding:
https://hg.mozilla.org/mozilla-central/rev/618652af63c9

Disable test:
https://hg.mozilla.org/mozilla-central/rev/ccf120bf4638
Whiteboard: [test disabled][leave open]
Target Milestone: --- → Firefox 17
(Assignee)

Comment 8

5 years ago
Spotted a regression from 783393. Need to fix this.
(Assignee)

Comment 9

5 years ago
Created attachment 655311 [details] [diff] [review]
v2

This should fix it.
Amongst other things, executeSoon is evil.
Attachment #655311 - Flags: review?(rcampbell)
Comment on attachment 655311 [details] [diff] [review]
v2

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

yuup.
Attachment #655311 - Flags: review?(rcampbell) → review+
(Assignee)

Comment 11

5 years ago
Green.
https://tbpl.mozilla.org/?tree=Try&rev=8f32b4f70832

Comment 12

5 years ago
https://hg.mozilla.org/integration/fx-team/rev/e11af99c5566

Updated

5 years ago
Whiteboard: [test disabled][leave open] → [test disabled][fixed-in-fx-team]
(Assignee)

Updated

5 years ago
Whiteboard: [test disabled][fixed-in-fx-team] → [fixed-in-fx-team]

Comment 13

5 years ago
https://hg.mozilla.org/mozilla-central/rev/e11af99c5566
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.