Closed
Bug 1147413
Opened 10 years ago
Closed 9 years ago
[e10s] browser_markupview_node_not_displayed_02.js causes unsafe CPOW usage warnings
Categories
(DevTools :: Inspector, defect)
DevTools
Inspector
Tracking
(e10s+)
RESOLVED
DUPLICATE
of bug 1252099
Tracking | Status | |
---|---|---|
e10s | + | --- |
People
(Reporter: mconley, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [unsafe-cpow-usage])
Mined from test logs
In devtools/markupview/test/browser_markupview_node_not_displayed_02.js:
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l17
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l18
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l19
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l20
{
desc: "Hiding a node by creating a new stylesheet",
selector: "#normal-div",
before: true,
changeStyle: (doc, node) => {
let div = doc.createElement("div"); <-- causes unsafe CPOW usage warning
div.id = "new-style"; <-- causes unsafe CPOW usage warning
div.innerHTML = "<style>#normal-div {display:none;}</style>"; <-- causes unsafe CPOW usage warning
doc.body.appendChild(div); <-- causes unsafe CPOW usage warning
},
after: false
},
...
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l29
{
desc: "Showing a node by deleting an existing stylesheet",
selector: "#normal-div",
before: false,
changeStyle: (doc, node) => {
doc.getElementById("new-style").remove(); <-- causes unsafe CPOW usage warning
},
after: true
},
...
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l38
{
desc: "Hiding a node by changing its style property",
selector: "#display-none",
before: false,
changeStyle: (doc, node) => {
node.style.display = "block"; <-- causes unsafe CPOW usage warning
},
after: true
},
...
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l47
{
desc: "Showing a node by removing its hidden attribute",
selector: "#hidden-true",
before: false,
changeStyle: (doc, node) => {
node.removeAttribute("hidden"); <-- causes unsafe CPOW usage warning
},
after: true
},
...
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l56
{
desc: "Hiding a node by adding a hidden attribute",
selector: "#hidden-true",
before: true,
changeStyle: (doc, node) => {
node.setAttribute("hidden", "true"); <-- causes unsafe CPOW usage warning
},
after: false
},
...
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l65
{
desc: "Showing a node by changin a stylesheet's rule",
selector: "#hidden-via-stylesheet",
before: false,
changeStyle: (doc, node) => {
doc.styleSheets[0].cssRules[0].style.setProperty("display", "inline"); <-- causes unsafe CPOW usage warning
},
after: true
},
...
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l74
{
desc: "Hiding a node by adding a new rule to a stylesheet",
selector: "#hidden-via-stylesheet",
before: true,
changeStyle: (doc, node) => {
doc.styleSheets[0].insertRule(
"#hidden-via-stylesheet {display: none;}", 1); <-- causes unsafe CPOW usage warning
},
after: false
},
...
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l84
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l86
{
desc: "Hiding a node by adding a class that matches an existing rule",
selector: "#normal-div",
before: true,
changeStyle: (doc, node) => {
doc.styleSheets[0].insertRule(
".a-new-class {display: none;}", 2); <-- causes unsafe CPOW usage warning
node.classList.add("a-new-class"); <-- causes unsafe CPOW usage warning
},
after: false
}
...
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/browser_markupview_node_not_displayed_02.js#l113
info("Listening for the display-change event");
let onDisplayChanged = promise.defer();
inspector.markup.walker.once("display-change", onDisplayChanged.resolve);
info("Making style changes");
changeStyle(content.document, getNode(selector)); <-- causes unsafe CPOW usage warning
let nodes = yield onDisplayChanged.promise;
info("Verifying that the list of changed nodes include our container");
ok(nodes.length, "The display-change event was received with a nodes");
Reporter | ||
Updated•9 years ago
|
tracking-e10s:
m8+ → ---
Whiteboard: [unsafe-cpow-usage]
Comment 2•9 years ago
|
||
Inspector bug triage. Filter on CLIMBING SHOES
Fixed by Bug 1252099.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•