Closed Bug 1570720 Opened 5 years ago Closed 5 years ago

DevTools CSS Rules View fails to update the page for my style changes, when I'm viewing about:logins (with error `TypeError: "decl is null"`)

Categories

(DevTools :: Inspector: Rules, defect)

Product:
DevTools
Component:
Inspector: Rules
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1391994

People

(Reporter: dholbert, Unassigned)

References

Details

STR:

  1. Visit the special internal page about:logins
  2. Right-click the icon at the top left and choose "Inspect Element"
  3. Focus the <body> or the <header> node in the inspector. (any node will do, I think)
  4. In the Rules pane, add "display:none" in the element {...} section at the top.
  5. (optional) Uncheck the declaration that you just added.

ACTUAL RESULTS:

  • The page never changes its rendering.
  • After step 4, I see style="" appear in the left-pane DOM view (notably, this new style attribute is empty)
  • After step 5, this error appears in the Ctrl+Shift+J Browser Console:
TypeError: "decl is null"
    setPropertyEnabled resource://devtools/shared/fronts/inspector/rule-rewriter.js:406
    setPropertyEnabled resource://devtools/client/inspector/rules/models/rule.js:556
    resultPromise resource://devtools/client/inspector/rules/models/rule.js:446
   [...]

EXPECTED RESULTS:

  • After step 4, you should see the content disappear, and you should see style="display:none" automatically added to the focused node in the left-pane DOM view.
  • After step 5, you should see the content reappear and there shouldn't be any errors in Browser Console.

Testing this, I see a CSP warning in the console:

Content Security Policy: The page’s settings blocked the loading of a resource at inline (“default-src”).

Makes me think that this is a duplicate of bug 1391994.

Daniel, do you agree that this is a duplicate?

Flags: needinfo?(dholbert)
See Also: → 1391994

Tested again, yes, this is definitely a CSP issue, and a duplicate of bug 1391994. So I'll close it now. We do need to fix that problem soon. It's starting to affect internal about pages more and more.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(dholbert)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.