Dispatch change event for input type=color when colorpicker is closed

LayoutTests/fast/forms/color/input-color-onchange-event.html

Index: LayoutTests/fast/forms/color/input-color-onchange-event.html
diff --git a/LayoutTests/fast/forms/color/input-color-onchange-event.html b/LayoutTests/fast/forms/color/input-color-onchange-event.html
index d0f53acdcf599021d928e8599eaec3144744d8a7..16cf2a21950f7a38b9f4f9ada46873c84bee2094 100644
--- a/LayoutTests/fast/forms/color/input-color-onchange-event.html
+++ b/LayoutTests/fast/forms/color/input-color-onchange-event.html
@@ -20,8 +20,11 @@ input.style.top = '0';
 input.style.width = '20px';
 input.style.height = '20px';
 
+var onChange = 0;
+
 input.onchange = function() {
     debug("value changed to " + input.value);
+    onChange++;
 };
 
 eventSender.mouseMoveTo(10, 10);
@@ -34,10 +37,19 @@ shouldThrow("internals.selectColorInColorChooser(document, '#ff0000');");
 
 // input.onchange should be called
 internals.selectColorInColorChooser(input, '#ff0000');
+internals.endColorChooser(input);

[keishi, 2014/09/17 08:15:54]

Ditto. No need to add this.

[Habib Virji, 2014/09/17 08:30:42]

Added as line 26 above, will not be printed about change event generation.

[keishi, 2014/09/18 03:50:55]

We don't have a test for input events so I think we could add that here.

input.oninput = function() {
  debug("input event dispatched: " + input.value);
};

On non-mac, an input event should only be fired for the first
internals.selectColorInColorChooser but not for the second one.

[Habib Virji, 2014/09/18 12:42:05]

Done.

 // input.onchange should not be called
 internals.selectColorInColorChooser(input, '#ff0000');
-
+internals.endColorChooser(input);
 shouldBe('input.value', '"#ff0000"');
+
+debug('Change event is only dispatched, when color chooser is closed');
+onChange = 0;
+internals.selectColorInColorChooser(input, '#ff0002');
+shouldBe('onChange', '0');
+internals.endColorChooser(input);
+shouldBe('onChange', '1');
+
 </script>
 </body>
 </html>