Closed Bug 1310337 Opened 8 years ago Closed 8 years ago

Stop using chrome urls in inspector stylesheets

Categories

(DevTools :: Inspector, defect)

defect
Not set
normal

Tracking

(firefox52 affected)

RESOLVED DUPLICATE of bug 1311541
Tracking Status
firefox52 --- affected

People

(Reporter: jdescottes, Unassigned)

References

Details

Attachments

(1 file)

Blocking Bug 1291049. To enable images from stylesheets when loading the inspector in html, the stylesheets used by the inspector should not rely on chrome URLs.
Comment on attachment 8801342 [details] Bug 1310337 - remove chrome urls from inspector stylesheets; https://reviewboard.mozilla.org/r/86122/#review84726 Clearing until the test issue I mentioned on IRC is fixed.
Attachment #8801342 - Flags: review?(ntim.bugs)
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Iteration: --- → 52.2 - Oct 17
Flags: qe-verify?
Priority: -- → P1
Whiteboard: [devtools-html]
While related to loading the inspector in a content tab, the impact of this bug is that images won't be displayed, but technically it doesn't prevent using the inspector. After reviewing the remaining bugs I still have to log, I think we should put this one in the reserve. Unassigning myself at the moment.
Assignee: jdescottes → nobody
Status: ASSIGNED → NEW
Priority: P1 → P3
Whiteboard: [devtools-html] → [reserve-html]
Iteration: 52.2 - Oct 17 → ---
Summary of our conversion on IRC. With the patch attached here the test at browser/base/content/test/general/browser_parsable_css.js will fail. We already have some images loaded via relative URLs in our stylesheets which are whitelisted in this test. See the discussion in Bug 1302708 for hints at how to handle this (load & package the images as resources apparently). The end goal of this bug remains to be able to have images when using the inspector in a content tab. If another solution (let the development server rewrite the stylesheet dynamically?) seems more appropriate and less impacting for the "real" devtools, we should also consider it.
Depends on: 1311541
Priority: P3 → P2
Whiteboard: [reserve-html] → [devtools-html]
This will be fully handled by Bug 1311541.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
No longer depends on: 1311541
No longer blocks: devtools-html-phase2
Flags: qe-verify?
Priority: P2 → --
Whiteboard: [devtools-html]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: