1147415 - [e10s] devtools/markupview/test/head.js causes unsafe CPOW usage warnings

Status: RESOLVED DUPLICATE of bug 1252099

(DevTools :: Inspector, defect)

Description

[Mike Conley (:mconley) (:⚙️)]

Mined from test logs

In devtools/markupview/test/head.js:

https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/head.js#l101

/**
 * Reload the current page
 * @return a promise that resolves when the inspector has emitted the event
 * new-root
 */
function reloadPage(inspector) {
  info("Reloading the page");
  let newRoot = inspector.once("new-root");
  content.location.reload(); <-- causes unsafe CPOW usage warning
  return newRoot;
}

...

https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/head.js#l134
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/head.js#l136
/**
 * Simple DOM node accesor function that takes either a node or a string css
 * selector as argument and returns the corresponding node
 * @param {String|DOMNode} nodeOrSelector
 * @return {DOMNode|CPOW} Note that in e10s mode a CPOW object is returned which
 * doesn't implement *all* of the DOMNode's properties
 */
function getNode(nodeOrSelector) {
  info("Getting the node for '" + nodeOrSelector + "'"); <-- causes unsafe CPOW usage warning
  return typeof nodeOrSelector === "string" ?
    content.document.querySelector(nodeOrSelector) : <-- causes unsafe CPOW usage warning
    nodeOrSelector;
}

...

https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/head.js#l319
https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/head.js#l322

function assertAttributes(selector, attrs) {
  let node = getNode(selector);

  is(node.attributes.length, Object.keys(attrs).length, <-- causes unsafe CPOW usage warning
    "Node has the correct number of attributes.");
  for (let attr in attrs) {
    is(node.getAttribute(attr), attrs[attr], <-- causes unsafe CPOW usage warning
      "Node has the correct " + attr + " attribute.");
  }
}

...

https://hg.mozilla.org/projects/holly/file/50e197279ad5/browser/devtools/markupview/test/head.js#l398

function wait(ms) {
  let def = promise.defer();
  content.setTimeout(def.resolve, ms); <-- causes unsafe CPOW usage warning
  return def.promise;
}

Comment 1

[Mike Conley (:mconley) (:⚙️)]

This is test only.

Comment 2

[Julian Descottes [:jdescottes]]

Inspector bug triage. Filter on CLIMBING SHOES

Fixed by Bug 1252099.