Blink does not respect input.selectionStart and input.selectionEnd for some cases

Source/core/html/forms/TextFieldInputType.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2011 Apple 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.
@@ skipping 141 matching lines @@
     // changes input type.
     RefPtr<HTMLInputElement> input(element());
 
     // We don't ask InputType::setValue to dispatch events because
     // TextFieldInputType dispatches events different way from InputType.
     InputType::setValue(sanitizedValue, valueChanged, DispatchNoEvent);
 
     if (valueChanged)
         input->updateView();
 
-    unsigned max = visibleValue().length();
-    if (input->focused())
+    int max = visibleValue().length();
+    if (input->focused()) {
         input->setSelectionRange(max, max);
-    else
-        input->cacheSelectionInResponseToSetValue(max);
+    } else {
+        if (max < input->selectionStart())

[tkent, 2014/05/19 00:00:34]

This |if| statement should be
    input->setSelectionRange(input->selectionStart(), input->selectionEnd());
and setSelectionRange should adjust the values.

[harpreet.sk, 2014/05/19 11:16:52]

Done.

+            input->setSelectionRange(max, max);
+        else if (max >= input->selectionStart() && max < input->selectionEnd())
+            input->setSelectionRange(input->selectionStart(), max);
+        else
+            input->setSelectionRange(input->selectionStart(), input->selectionEnd());
+    }
 
     if (!valueChanged)
         return;
 
     switch (eventBehavior) {
     case DispatchChangeEvent:
         // If the user is still editing this field, dispatch an input event rather than a change event.
         // The change event will be dispatched when editing finishes.
         if (input->focused())
             input->dispatchFormControlInputEvent();
@@ skipping 376 matching lines @@
     return shouldSpinButtonRespondToMouseEvents() && element().focused();
 }
 
 void TextFieldInputType::spinButtonDidReleaseMouseCapture(SpinButtonElement::EventDispatch eventDispatch)
 {
     if (eventDispatch == SpinButtonElement::EventDispatchAllowed)
         element().dispatchFormControlChangeEvent();
 }
 
 } // namespace WebCore