Disable Style Inspector from Highlighter UI

RESOLVED FIXED in Firefox 9

Status

()

Firefox
Developer Tools
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: rc, Assigned: rc)

Tracking

unspecified
Firefox 9
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Ensure that the Style Inspector is disabled in the Highlighter UI until the leaks described in bug 689160 are fixed.
(Assignee)

Comment 1

6 years ago
Created attachment 562433 [details] [diff] [review]
[in-fx-team] Disable Style Inspector
Assignee: nobody → rcampbell
Status: NEW → ASSIGNED
Attachment #562433 - Flags: review?(mihai.sucan)
(Assignee)

Comment 2

6 years ago
Without Patch (and bug 663831 and associates applied):

    --DOMWINDOW == 41 (0x159a29dc8) [serial = 17] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 40 (0x160996bc8) [serial = 22] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 39 (0x1638e9c98) [serial = 28] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 38 (0x1692832c8) [serial = 32] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 37 (0x16b3b3878) [serial = 39] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 36 (0x141850938) [serial = 47] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 35 (0x1072dea18) [serial = 3] [outer = 0x0] [url = chrome://browser/content/hiddenWindow.xul]
    --DOMWINDOW == 34 (0x10723a108) [serial = 1] [outer = 0x0] [url = chrome://browser/content/browser.xul]
    --DOMWINDOW == 33 (0x13ea54148) [serial = 51] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 18 (0x141804848) [serial = 77] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 17 (0x13eaec5b8) [serial = 69] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 16 (0x1418ccec8) [serial = 105] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 15 (0x1418979b8) [serial = 91] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 14 (0x16b3cbcb8) [serial = 100] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 13 (0x16d04e1a8) [serial = 88] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 12 (0x16d04a6f8) [serial = 87] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 11 (0x16b305f78) [serial = 35] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 10 (0x1528c1438) [serial = 54] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 9 (0x16b3c9bb8) [serial = 59] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 8 (0x1609d0a08) [serial = 23] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 7 (0x141861c78) [serial = 48] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 6 (0x16d71aa38) [serial = 29] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 5 (0x141ec8268) [serial = 18] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 4 (0x12cd95908) [serial = 13] [outer = 0x0] [url = about:blank]
    --DOMWINDOW == 3 (0x16b3df1a8) [serial = 57] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 2 (0x16b3bb048) [serial = 40] [outer = 0x0] [url = chrome://browser/content/csshtmltree.xhtml]
    --DOMWINDOW == 1 (0x1072e1c58) [serial = 4] [outer = 0x0] [url = chrome://browser/content/hiddenWindow.xul]
    --DOMWINDOW == 0 (0x107246058) [serial = 2] [outer = 0x0] [url = about:blank] 

With patch applied:

--DOCSHELL 0x12a840160 == 2
--DOMWINDOW == 11 (0x12cff4498) [serial = 14] [outer = 0x12ccb4770] [url = about:blank]
--DOMWINDOW == 10 (0x12cf57c18) [serial = 10] [outer = 0x12a8c5e00] [url = about:blank]
--DOMWINDOW == 9 (0x12a8c5e78) [serial = 6] [outer = 0x0] [url = about:blank]
--DOMWINDOW == 8 (0x12cf2fdf8) [serial = 9] [outer = 0x12a8ae3e0] [url = about:blank]
--DOMWINDOW == 7 (0x12a8ae458) [serial = 5] [outer = 0x0] [url = about:blank]
Depends on: 663831
Comment on attachment 562433 [details] [diff] [review]
[in-fx-team] Disable Style Inspector

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

Patch looks fine. All tests pass locally.
Attachment #562433 - Flags: review?(mihai.sucan) → review+
(Assignee)

Updated

6 years ago
Whiteboard: [fixed-in-fx-team]
(Assignee)

Comment 4

6 years ago
Comment on attachment 562433 [details] [diff] [review]
[in-fx-team] Disable Style Inspector

https://hg.mozilla.org/integration/fx-team/rev/381904af2863
Attachment #562433 - Attachment description: Disable Style Inspector → [in-fx-team] Disable Style Inspector
https://hg.mozilla.org/mozilla-central/rev/381904af2863
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 9
You need to log in before you can comment on or make changes to this bug.