Closed Bug 855799 Opened 11 years ago Closed 11 years ago

Starting customization before panel opens causes customization not to work.

Categories

(Firefox :: Toolbars and Customization, defect)

x86_64
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 28

People

(Reporter: mconley, Assigned: mconley)

References

Details

Attachments

(1 file)

STR:

Using Jamun Nightly (https://hg.mozilla.org/projects/jamun/rev/0362bdd57830) on Windows 7.

1) Open Firefox
2) Go to AppMenu > Options > Toolbar Layout to enter Customization Mode

What happens?

Customization mode is entered, but the menu panel is empty, and no items are in the palette. The error console says:

Timestamp: 28/03/2013 2:54:15 PM
Error: Error: No build area nodes registered for PanelUI-contents
Source File: resource:///modules/CustomizableUI.jsm
Line: 1348

This is because the panel lazily registers itself as a build area after it is first opened. In the STR above, the panel never got a chance to register itself.
Attached patch Patch v1Splinter Review
The panel is special because customization mode *assumes* that the panel is available. We might have to do something similar and more generic for other lazy areas down the road, but until we get there, I think this should suffice.
Assignee: nobody → mconley
Attachment #730807 - Flags: review?(jAwS)
Attachment #730807 - Flags: review?(jAwS) → review+
I think it's OK to wait for this to land on UX in milestone 2.
Blocks: 855287
No longer blocks: 770135
Landed on Jamun as https://hg.mozilla.org/projects/jamun/rev/2c77733f1625
OS: Windows 7 → All
Whiteboard: [fixed in jamun]
Status: NEW → ASSIGNED
Whiteboard: [fixed in jamun] → [fixed in jamun][fixed in ux]
https://hg.mozilla.org/mozilla-central/rev/f01490f98a92
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed in jamun][fixed in ux]
Target Milestone: --- → Firefox 28
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: