Closed Bug 1665341 Opened 4 years ago Closed 4 years ago

Remove BoxModelRenderer

Categories

(DevTools :: Inspector, task)

task

Tracking

(firefox82 fixed)

RESOLVED FIXED
82 Branch
Tracking Status
firefox82 --- fixed

People

(Reporter: rcaliman, Assigned: rcaliman)

References

Details

Attachments

(1 file)

BoxModelRenderer was added as a standalone module that only renders the Box Model Highlighter. It was part of a previous attempt (Bug 1572651) at making highlighters work across processes via message manager. It was kept around for another attempt that involved rendering highlighters in the parent process (Bug 1598307). That effort was ultimately superseded by a client-side implementation to toggle highlighter instances (Bug 1646028). This final approach uses the all-in-one BoxModelHighlighter class, thus making the standalone renderer part, BoxModelRenderer obsolete.

Assignee: nobody → rcaliman
Status: NEW → ASSIGNED
Depends on: 1646028
Pushed by rcaliman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cc3404de8f4d
Remove obsolete BoxModelRenderer. r=jdescottes
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
Component: Inspector: Highlighters → Inspector
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: