'dbg close' errors when toolbox is already closed

RESOLVED FIXED in Firefox 24

Status

()

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

People

(Reporter: anton, Assigned: anton)

Tracking

Trunk
Firefox 24
x86
Mac OS X
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Steps to reproduce:

1. Close toolbox.
2. Open GCLI.
3. Run 'dbg close'

It will error, it shouldn't.
(Assignee)

Comment 1

5 years ago
Created attachment 764509 [details] [diff] [review]
Make 'dbg close' do nothing when toolbox is closed

Patch similar to the one for 'profiler close'
Attachment #764509 - Flags: review?(jwalker)
Attachment #764509 - Flags: review?(dcamp)
(Assignee)

Comment 2

5 years ago
Created attachment 764517 [details] [diff] [review]
Make 'dbg close' do nothing when toolbox is closed

Previous patch had a small typo.
Attachment #764509 - Attachment is obsolete: true
Attachment #764509 - Flags: review?(jwalker)
Attachment #764509 - Flags: review?(dcamp)
Attachment #764517 - Flags: review?(jwalker)
Attachment #764517 - Flags: review?(dcamp)
Comment on attachment 764517 [details] [diff] [review]
Make 'dbg close' do nothing when toolbox is closed

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

I think my r+ can do for dcamp too.

The margin for improvement in these tests is small to non-existant, but 2 facts that might not be obvious about helpers:

* helpers.audit returns a promise that is only completed when it is totally done
* There is a 'post' property that lets you run tests to check that everything worked ok
  So ...

    return helpers.check([{
      setup: 'profiler close',
      exec: {},
      post: function() {
        return getPromiseFulfilledWhenTheProfilerCloses()
      }
    }]);

The need to use events makes it hard to use this style, but I thought it was worth noting in case it wasn't obvious.
Attachment #764517 - Flags: review?(jwalker) → review+
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/integration/fx-team/rev/437caa035564
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/437caa035564
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 24

Updated

4 years ago
Attachment #764517 - Flags: review?(dcamp)
You need to log in before you can comment on or make changes to this bug.