Closed Bug 597150 Opened 11 years ago Closed 10 years ago

Inconsistent usage of HUDService.registerActiveContext()

Categories

(DevTools :: Console, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX
Firefox 15

People

(Reporter: msucan, Assigned: msucan)

References

Details

(Whiteboard: [post-fx-4] [console-1][resolved by bug 673148])

HUDService.activateHUDForContext(tab) calls registerActiveContext(notificationBox.id), while other code uses hudId for (un)registerActiveContext(). This needs fixing, before something bad breaks.
Whiteboard: [post-fx-4]
I think a re-naming will fix this, no?
Whiteboard: [post-fx-4] → [post-fx-4] [console-1]
We'll need to recheck the code. There have been many changes to that stuff - most likely this bug is no longer valid.
Component: Developer Tools → Developer Tools: Console
QA Contact: developer.tools → developer.tools.console
The patch for bug 673148 is likely to makes this bug invalid as it removes the registerActiveContext method.
Depends on: async-webconsole
Assignee: nobody → mihai.sucan
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Whiteboard: [post-fx-4] [console-1] → [post-fx-4] [console-1][resolved by bug 673148]
Target Milestone: --- → Firefox 15
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.