598270 - HUD should keep a reference to its <browser>

Status: RESOLVED FIXED

(DevTools :: Console, defect)

Description

[Justin Dolske [:Dolske]]

Gavin made a frowny face in bug 596371 comment 11:

>diff --git a/toolkit/components/console/hudservice/HUDService.jsm b/toolkit/components/console/hudservice/HUDService.jsm

>   getContentWindowFromHUDId: function HS_getContentWindowFromHUDId(aHUDId)

>+      var node = nodes[i];
>+
>+      if (node.localName == "stack" &&
>+          node.firstChild &&
>+          node.firstChild.contentWindow) {
>+        return node.firstChild.contentWindow;

so ugly :( "hud" objects should really have direct references to their
associated <browser>. The whole HUD object hierarchy is horribly confusing
though. File a followup?

Comment 1

[David Dahl :ddahl]

(In reply to comment #0)
> Gavin made a frowny face in bug 596371 comment 11:
 
> so ugly :( "hud" objects should really have direct references to their
> associated <browser>. The whole HUD object hierarchy is horribly confusing
> though. File a followup?

indeed it is. We have been gun shy about holding too many references in hopes of not introducing leaks. Please file a followup. We would like to clean and straighten out some of this gunk post fx-4.

Comment 2

[Rob Campbell [:rc] (:robcee)]

I am pretty sure this was fixed a very long time ago.