Closed Bug 1569495 Opened 5 years ago Closed 5 years ago

Stop using inline styles and innerHTML in devtools ImageTooltipHelper

Categories

(DevTools :: about:debugging, task, P3)

task

Tracking

(firefox70 fixed)

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: ckerschb, Assigned: jdescottes)

References

Details

Attachments

(2 files)

Within Bug 1567877 we are applying a meta CSP to about:devtools-toolbox. While that mostly works fine there is inline style code within ImageTooltipHelper.js [1] which needs to be refactored so we can remove the 'unsafe-inline' from the CSP. It's not a blocker for Bug 1567877 because we mostly want to avoid inline scripts. It would be good if we can get 'unsafe-inline' removed however.

Additionally that code uses innerHTML which we also want to avoid. So now there are already 2 reasons to refactor that piece of code :-)

[1] https://searchfox.org/mozilla-central/rev/5e660d3dfcba897c8501e3fda1d415565a096e7e/devtools/client/shared/widgets/tooltip/ImageTooltipHelper.js#117

Depends on D39644

Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/531fa7ad6b16 Stop using inline styles and inner HTML for image tooltip r=miker https://hg.mozilla.org/integration/autoland/rev/6712b3dd77fc Stop using innerHTML in HTML Tooltip r=miker
Blocks: 1567877
No longer depends on: 1567877
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Summary: Remove 'unsafe-inline' from the CSP of about:devtools-toolbox → Stop using inline styles and innerHTML in devtools ImageTooltipHelper
Priority: -- → P3
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: