Value set in onblur should not keep element in focus

Source/core/html/shadow/DateTimeFieldElement.cpp

 /*
  * Copyright (C) 2012 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:
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *     notice, this list of conditions and the following disclaimer in the
@@ skipping 36 matching lines @@
 }
 
 DateTimeFieldElement::DateTimeFieldElement(Document& document, FieldOwner& fieldOwner)
     : HTMLSpanElement(document)
     , m_fieldOwner(&fieldOwner)
 {
 }
 
 void DateTimeFieldElement::defaultEventHandler(Event* event)
 {
-    if (event->type() == EventTypeNames::blur)
-        didBlur();
-
-    if (event->type() == EventTypeNames::focus)
-        didFocus();
-
     if (event->isKeyboardEvent()) {
         KeyboardEvent* keyboardEvent = toKeyboardEvent(event);
         if (!isDisabled() && !isFieldOwnerDisabled() && !isFieldOwnerReadOnly()) {
             handleKeyboardEvent(keyboardEvent);
             if (keyboardEvent->defaultHandled())
                 return;
         }
         defaultKeyboardEventHandler(keyboardEvent);
         if (keyboardEvent->defaultHandled())
             return;
@@ skipping 49 matching lines @@
         return;
     }
 
     if (keyIdentifier == "U+0008" || keyIdentifier == "U+007F") {
         keyboardEvent->setDefaultHandled();
         setEmptyValue(DispatchEvent);
         return;
     }
 }
 
-void DateTimeFieldElement::didBlur()
+void DateTimeFieldElement::setFocus(bool value)
 {
     if (m_fieldOwner)
-        m_fieldOwner->didBlurFromField();
-}
-
-void DateTimeFieldElement::didFocus()
-{
-    if (m_fieldOwner)
-        m_fieldOwner->didFocusOnField();
+        value ? m_fieldOwner->didFocusOnField():m_fieldOwner->didBlurFromField();

[tkent, 2014/03/31 04:17:08]

nit: Needs spaces around ':'.

[Habib Virji, 2014/03/31 09:05:51]

Done.

+    ContainerNode::setFocus(value);
 }
 
 void DateTimeFieldElement::focusOnNextField()
 {
     if (!m_fieldOwner)
         return;
     m_fieldOwner->focusOnNextField(*this);
 }
 
 void DateTimeFieldElement::initialize(const AtomicString& pseudo, const String& axHelpText, int axMinimum, int axMaximum)
@@ skipping 80 matching lines @@
 }
 
 int DateTimeFieldElement::valueForARIAValueNow() const
 {
     return valueAsInteger();
 }
 
 } // namespace WebCore
 
 #endif