Bug 1365053 - called `_onEyeDropperDone` once the color is selected; r=jdescottes
MozReview-Commit-ID: JUppfvNYmTJ
--- a/devtools/client/inspector/rules/test/browser_rules_eyedropper.js
+++ b/devtools/client/inspector/rules/test/browser_rules_eyedropper.js
@@ -96,16 +96,19 @@ function* testSelect(view, swatch, inspe
options: {type: "mouseup"}});
yield onPicked;
yield onRuleViewChanged;
let color = swatch.style.backgroundColor;
is(color, EXPECTED_COLOR, "swatch changed colors");
+ ok(!swatch.eyedropperOpen, "swatch eye dropper is closed");
+ ok(!swatch.activeSwatch, "no active swatch");
+
is((yield getComputedStyleProperty("div", null, "background-color")),
EXPECTED_COLOR,
"div's color set to body color after dropper");
}
function* openEyedropper(view, swatch) {
let tooltip = view.tooltips.getTooltip("colorPicker").tooltip;
--- a/devtools/client/shared/widgets/tooltip/SwatchColorPickerTooltip.js
+++ b/devtools/client/shared/widgets/tooltip/SwatchColorPickerTooltip.js
@@ -171,16 +171,17 @@ SwatchColorPickerTooltip.prototype = Her
this.hide();
this.tooltip.emit("eyedropper-opened");
}, e => console.error(e));
inspector.once("color-picked", color => {
toolbox.win.focus();
this._selectColor(color);
+ this._onEyeDropperDone();
});
inspector.once("color-pick-canceled", () => {
this._onEyeDropperDone();
});
},
_onEyeDropperDone: function () {