58 bytes, text/x-review-board-request
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)
Iteration: --- → 52.2 - Oct 17
Priority: -- → P1
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]
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.
Priority: P3 → P2
Whiteboard: [reserve-html] → [devtools-html]
This will be fully handled by Bug 1311541.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1311541
No longer blocks: devtools-html-phase2
Priority: P2 → --
You need to log in before you can comment on or make changes to this bug.