1146650 - [e10s] browser_markupview_html_edit_01.js causes unsafe CPOW usage warnings

Status: RESOLVED DUPLICATE of bug 1252099

(DevTools :: Inspector, defect)

Description

[Ian Moody [:Kwan] (UTC+1)]

Mined from test logs

In browser/devtools/markupview/test/browser_markupview_html_edit_01.js:

  {
    selector: "#one",
    oldHTML: '<div id="one">First <em>Div</em></div>',
    newHTML: '<div id="one">First Div</div>',
    validate: function*(pageNode, pageNodeFront, selectedNodeFront) {
      is(pageNode.textContent, "First Div", "New div has expected text content"); <- Causes CPOW warning
      ok(!getNode("#one em"), "No em remaining")
    }
  },
[...]
  {
    selector: "#addedAttribute",
    oldHTML: '<div id="addedAttribute">addedAttribute</div>',
    newHTML: '<div id="addedAttribute" class="important" disabled checked>addedAttribute</div>',
    validate: function*(pageNode, pageNodeFront, selectedNodeFront) {
      is(pageNodeFront, selectedNodeFront, "Original element is selected");
      is(pageNode.outerHTML, '<div id="addedAttribute" class="important" disabled="" checked="">addedAttribute</div>', <- Causes CPOW warning
            "Attributes have been added");
    }
  },
[...]
  {
    selector: "#siblings",
    oldHTML: '<div id="siblings">siblings</div>',
    newHTML: '<div id="siblings-before-sibling">before sibling</div>' +
             '<div id="siblings">siblings (updated)</div>' +
             '<div id="siblings-after-sibling">after sibling</div>',
    validate: function*(pageNode, pageNodeFront, selectedNodeFront, inspector) {
      let beforeSibling = getNode("#siblings-before-sibling");
      let beforeSiblingFront = yield getNodeFront("#siblings-before-sibling", inspector);
      let afterSibling = getNode("#siblings-after-sibling");

      is(beforeSiblingFront, selectedNodeFront, "Sibling has been selected");
      is(pageNode.textContent, "siblings (updated)", "New div has expected text content"); <- Causes CPOW warning
      is(beforeSibling.textContent, "before sibling", "Sibling has been inserted"); <- Causes CPOW warning
      is(afterSibling.textContent, "after sibling", "Sibling has been inserted"); <- Causes CPOW warning
    }
  }

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.