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

Source/core/html/HTMLInputElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2007 Samuel Weinig (sam@webkit.org)
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2008 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2012 Samsung Electronics. All rights reserved.
@@ skipping 1464 matching lines @@
 }
 
 void HTMLInputElement::selectColorInColorChooser(const Color& color)
 {
     // FIXME: Remove type check and static_cast.
     if (type() != InputTypeNames::color)
         return;
     static_cast<ColorInputType*>(m_inputType.get())->didChooseColor(color);
 }
 
+void HTMLInputElement::endColorChooser()
+{
+    // FIXME: Remove type check and static_cast.
+    if (type() != InputTypeNames::color)

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

We decided not to add any more if statements that check the type to
HTMLInputElement, and instead use InputType/InputTypeView.
Could you add a method like InputType::endColorChooser() and a comment saying
that it is for testing purposes only.

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

Ok, will make a change and push in new patch. 

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

Done.

+        return;
+    static_cast<ColorInputType*>(m_inputType.get())->didEndChooser();
+}
+
 HTMLElement* HTMLInputElement::list() const
 {
     return dataList();
 }
 
 HTMLDataListElement* HTMLInputElement::dataList() const
 {
     if (!m_hasNonEmptyList)
         return 0;
 
@@ skipping 299 matching lines @@
 {
     listAttributeTargetChanged();
 }
 
 AXObject* HTMLInputElement::popupRootAXObject()
 {
     return m_inputTypeView->popupRootAXObject();
 }
 
 } // namespace