Closed Bug 852110 Opened 11 years ago Closed 11 years ago

Allow tools to build a concrete panel rather than only a promise of a panel

Categories

(DevTools :: Framework, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 22

People

(Reporter: jwalker, Assigned: jwalker)

Details

Attachments

(1 file)

i.e. Allow:

    build: function(frame, target) {
      return new MyPanel(frame, target);
    }

In addition to

    build: function(frame, target) {
      var panel = new MyPanel(frame, target);
      return panel.open(window);
    }
    MyPanel.prototype.open = function() {
      return Promise.resolve(this);
    };
Attached patch v1Splinter Review
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Attachment #726140 - Flags: review?(mratcliffe)
Comment on attachment 726140 [details] [diff] [review]
v1

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

Simple enough.
Attachment #726140 - Flags: review?(mratcliffe) → review+
https://hg.mozilla.org/mozilla-central/rev/ed8fc1dfe8aa
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 22
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: