Closed
Bug 597150
Opened 15 years ago
Closed 13 years ago
Inconsistent usage of HUDService.registerActiveContext()
Categories
(DevTools :: Console, defect)
DevTools
Console
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.
|
||
Updated•15 years ago
|
Whiteboard: [post-fx-4]
|
|
||
Comment 1•14 years ago
|
||
I think a re-naming will fix this, no?
Whiteboard: [post-fx-4] → [post-fx-4] [console-1]
|
Assignee | |
Comment 2•14 years ago
|
||
We'll need to recheck the code. There have been many changes to that stuff - most likely this bug is no longer valid.
|
||
Updated•13 years ago
|
Component: Developer Tools → Developer Tools: Console
QA Contact: developer.tools → developer.tools.console
|
|
||
Comment 3•13 years ago
|
||
The patch for bug 673148 is likely to makes this bug invalid as it removes the registerActiveContext method.
Depends on: async-webconsole
|
|
||
Updated•13 years ago
|
Blocks: consolecleanup
|
|
Assignee | |
Updated•13 years ago
|
Assignee: nobody → mihai.sucan
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Whiteboard: [post-fx-4] [console-1] → [post-fx-4] [console-1][resolved by bug 673148]
Target Milestone: --- → Firefox 15
|
||
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•