Add SelectionMode enum for setRangeText() in HTMLInputElement and HTMLTextAreaElement

Add SelectionMode enum for setRangeText() in HTMLInputElement and HTMLTextAreaElement

In FF invalid SelectionMode values throw TypeError
In IE 'setRangeText' method is not supported

BUG=508304
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=199316

[tanay.c, 2015-07-21 05:44:50]

tanay.c@samsung.com changed reviewers:
+ philipj@opera.com

[tanay.c, 2015-07-21 05:44:51]

Please have a look.

[tanay.c, 2015-07-21 05:45:20]

Please have a look.

[philipj_slow, 2015-07-21 07:34:21]

Please test this in Firefox and IE and document their behavior in the
description, making sure to also test case sensitivity. I would guess that
Firefox throws TypeError while IE does not, probably not caring about case
either, but I haven't actually tested.

Please also update HTMLTextFormControlElement::setRangeText to do case-sensitive
matching and replace the comment with ASSERT(selectionMode == "preserve").

[tanay.c, 2015-07-21 10:53:26]

On 2015/07/21 07:34:21, philipj wrote:
> Please test this in Firefox and IE and document their behavior in the
> description, making sure to also test case sensitivity. I would guess that
> Firefox throws TypeError while IE does not, probably not caring about case
> either, but I haven't actually tested.
> 
> Please also update HTMLTextFormControlElement::setRangeText to do
case-sensitive
> matching and replace the comment with ASSERT(selectionMode == "preserve").

Thanks for the comments Philip.
I have updated the HTMLTextFormControlElement::setRangeText method. 
In FF the TypeError exception is thrown for invalid argument types and 
it is case sensitive i.e TypeError thrown for 'PRESERVE' , 'Preserve' etc.

In IE the 'setTextRange' function does not seem to be supported. I get a 
SCRIPT438: Object doesn't support property or method 'setRangeText' error.

Please let me know any further comments.

[philipj_slow, 2015-07-21 11:18:58]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-07-21 11:19:03]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1229133004/40001

[commit-bot: I haz the power, 2015-07-21 12:38:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-21 12:38:42]

Dry run: Try jobs failed on following builders:
  win_blink_compile_dbg on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/win_blink_compile_dbg/bu...)

[philipj_slow, 2015-07-21 12:55:06]

Looks like there are some real test failures, please investigate :)

[tanay.c, 2015-07-22 13:41:17]

On 2015/07/21 12:55:06, philipj wrote:
> Looks like there are some real test failures, please investigate :)

Fixed the trybot failures. Please review.

[philipj_slow, 2015-07-22 14:43:53]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-07-22 14:43:54]

lgtm

[commit-bot: I haz the power, 2015-07-22 14:44:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1229133004/60001

[commit-bot: I haz the power, 2015-07-22 16:07:55]

Committed patchset #4 (id:60001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=199316