Bug 1454591 part 4 - Have devtools test not rely on font-family being the first longhand of font shorthand. r?tromey
MozReview-Commit-ID: 2URKcAOcCdV
--- a/devtools/client/inspector/shared/test/browser_styleinspector_tooltip-shorthand-fontfamily.js
+++ b/devtools/client/inspector/shared/test/browser_styleinspector_tooltip-shorthand-fontfamily.js
@@ -36,18 +36,25 @@ async function testRuleView(ruleView, no
// Get the computed font family property inside the font rule view
let propertyList = ruleView.element
.querySelectorAll(".ruleview-propertylist");
let fontExpander = propertyList[1].querySelectorAll(".ruleview-expander")[0];
fontExpander.click();
let rule = getRuleViewRule(ruleView, "#testElement");
- let valueSpan = rule
- .querySelector(".ruleview-computed .ruleview-propertyvalue");
+ let computedlist = rule.querySelectorAll(".ruleview-computed");
+ let valueSpan;
+ for (let computed of computedlist) {
+ let propertyName = computed.querySelector(".ruleview-propertyname");
+ if (propertyName.textContent == "font-family") {
+ valueSpan = computed.querySelector(".ruleview-propertyvalue");
+ break;
+ }
+ }
// And verify that the tooltip gets shown on this property
let previewTooltip = await assertShowPreviewTooltip(ruleView, valueSpan);
let images = panel.getElementsByTagName("img");
is(images.length, 1, "Tooltip contains an image");
ok(images[0].getAttribute("src")
.startsWith("data:"), "Tooltip contains a data-uri image as expected");