Update of change event for datetime input type

Source/core/html/forms/BaseMultipleFieldsDateAndTimeInputType.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 169 matching lines @@
     RefPtr<HTMLInputElement> input(element());
     String oldValue = input->value();
     String newValue = sanitizeValue(dateTimeEditElement()->value());
     // Even if oldValue is null and newValue is "", we should assume they are same.
     if ((oldValue.isEmpty() && newValue.isEmpty()) || oldValue == newValue) {
         input->setNeedsValidityCheck();
     } else {
         input->setValueInternal(newValue, DispatchNoEvent);
         input->setNeedsStyleRecalc();
         input->dispatchFormControlInputEvent();
-        input->dispatchFormControlChangeEvent();
     }
     input->notifyFormStateChanged();
     input->updateClearButtonVisibility();
 }
 
 bool BaseMultipleFieldsDateAndTimeInputType::hasCustomFocusLogic() const
 {
     return false;
 }
 
@@ skipping 51 matching lines @@
         return;
     }
 
     DateTimeEditElement* edit = this->dateTimeEditElement();
     if (!edit)
         return;
     DateComponents date;
     unsigned end;
     if (date.parseDate(value, 0, end) && end == value.length())
         edit->setOnlyYearMonthDay(date);
+    element().dispatchFormControlChangeEvent();
 }
 
 void BaseMultipleFieldsDateAndTimeInputType::pickerIndicatorChooseValue(double value)
 {
     ASSERT(std::isfinite(value) || std::isnan(value));
     if (std::isnan(value))
         element().setValue(emptyString(), DispatchInputAndChangeEvent);
     else
         element().setValueAsNumber(value, ASSERT_NO_EXCEPTION, DispatchInputAndChangeEvent);
 }
@@ skipping 151 matching lines @@
 void BaseMultipleFieldsDateAndTimeInputType::handleKeydownEvent(KeyboardEvent* event)
 {
     if (m_pickerIndicatorIsVisible
         && ((event->keyIdentifier() == "Down" && event->getModifierState("Alt")) || (RenderTheme::theme().shouldOpenPickerWithF4Key() && event->keyIdentifier() == "F4"))) {
         if (PickerIndicatorElement* element = pickerIndicatorElement())
             element->openPopup();
         event->setDefaultHandled();
     } else {
         forwardEvent(event);
     }
+    element().dispatchFormControlChangeEvent();

[tkent, 2014/01/20 02:05:22]

We shouldn't put dispatchFormControlChangeEvent here because it looks unclear
why we need it. We should move it elsewhere.

 }
 
 bool BaseMultipleFieldsDateAndTimeInputType::hasBadInput() const
 {
     DateTimeEditElement* edit = dateTimeEditElement();
     return element().value().isEmpty() && edit && edit->anyEditableFieldsHaveValues();
 }
 
 AtomicString BaseMultipleFieldsDateAndTimeInputType::localeIdentifier() const
 {
@@ skipping 159 matching lines @@
         clearButton->setInlineStyleProperty(CSSPropertyPointerEvents, CSSValueNone);
     } else {
         clearButton->removeInlineStyleProperty(CSSPropertyOpacity);
         clearButton->removeInlineStyleProperty(CSSPropertyPointerEvents);
     }
 }
 
 } // namespace WebCore
 
 #endif