Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change it in response to a specification change, and to improve compatibility with
Firefox and Edge.

Old behavior:
 CRLF, CR, or LF is counted as 2 because we limited the submission value.

New behavior:
 CRLF, CR, or LF is counted as 1 because we limit the API value.

We can use HTMLTextAreaElement::value().length() as is because it's the API value,
and other string input should be counted by computeLengthForAPIValue().

Approved intent-to-ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/MkSfvCan7kY/PgniO8jwAAAJ

BUG=624361
Committed: https://crrev.com/a15474ff7adff41a8956ab436ac63a7d5b7a090f
Cr-Commit-Position: refs/heads/master@{#421445}

[tkent, 2016-09-28 01:24:37]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-28 01:25:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2016-09-28 01:25:22]

Description was changed from

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

BUG=
==========

to

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

BUG=624361
==========

[tkent, 2016-09-28 01:31:37]

Description was changed from

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

BUG=624361
==========

to

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change it in response to a specification change, and to improve compatibility
with
Firefox and Edge.

Old behavior:
 CRLF, CR, or LF is counted as 2 because we limit the submission value.

New behavior:
 CRLF, CR, or LF is counted as 1 because we limit the API value.

We can use HTMLTextAreaElement::value().length() as is because it's the API
value,
and other string input should be counted by computeLengthForAPIValue().

BUG=624361
==========

[commit-bot: I haz the power, 2016-09-28 02:15:42]

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

[commit-bot: I haz the power, 2016-09-28 02:15:43]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[tkent, 2016-09-28 02:19:43]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-28 02:19:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-28 03:43:00]

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

[commit-bot: I haz the power, 2016-09-28 03:43:01]

Dry run: This issue passed the CQ dry run.

[tkent, 2016-09-28 03:48:49]

Description was changed from

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change it in response to a specification change, and to improve compatibility
with
Firefox and Edge.

Old behavior:
 CRLF, CR, or LF is counted as 2 because we limit the submission value.

New behavior:
 CRLF, CR, or LF is counted as 1 because we limit the API value.

We can use HTMLTextAreaElement::value().length() as is because it's the API
value,
and other string input should be counted by computeLengthForAPIValue().

BUG=624361
==========

to

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change it in response to a specification change, and to improve compatibility
with
Firefox and Edge.

Old behavior:
 CRLF, CR, or LF is counted as 2 because we limit the submission value.

New behavior:
 CRLF, CR, or LF is counted as 1 because we limit the API value.

We can use HTMLTextAreaElement::value().length() as is because it's the API
value,
and other string input should be counted by computeLengthForAPIValue().

Approved intent-to-ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/MkSfvCan7kY/PgniO8jw...


BUG=624361
==========

[tkent, 2016-09-28 03:48:56]

Patchset #1 (id:1) has been deleted

[tkent, 2016-09-28 03:49:41]

tkent@chromium.org changed reviewers:
+ keishi@chromium.org

[tkent, 2016-09-28 03:49:42]

keishi, would you review this please?

[keishi, 2016-09-28 04:10:13]

LGTM

[tkent, 2016-09-28 04:15:09]

Description was changed from

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change it in response to a specification change, and to improve compatibility
with
Firefox and Edge.

Old behavior:
 CRLF, CR, or LF is counted as 2 because we limit the submission value.

New behavior:
 CRLF, CR, or LF is counted as 1 because we limit the API value.

We can use HTMLTextAreaElement::value().length() as is because it's the API
value,
and other string input should be counted by computeLengthForAPIValue().

Approved intent-to-ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/MkSfvCan7kY/PgniO8jw...


BUG=624361
==========

to

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change it in response to a specification change, and to improve compatibility
with
Firefox and Edge.

Old behavior:
 CRLF, CR, or LF is counted as 2 because we limited the submission value.

New behavior:
 CRLF, CR, or LF is counted as 1 because we limit the API value.

We can use HTMLTextAreaElement::value().length() as is because it's the API
value,
and other string input should be counted by computeLengthForAPIValue().

Approved intent-to-ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/MkSfvCan7kY/PgniO8jw...


BUG=624361
==========

[tkent, 2016-09-28 04:16:31]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-09-28 04:16:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-28 04:21:35]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-09-28 04:24:03]

Description was changed from

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change it in response to a specification change, and to improve compatibility
with
Firefox and Edge.

Old behavior:
 CRLF, CR, or LF is counted as 2 because we limited the submission value.

New behavior:
 CRLF, CR, or LF is counted as 1 because we limit the API value.

We can use HTMLTextAreaElement::value().length() as is because it's the API
value,
and other string input should be counted by computeLengthForAPIValue().

Approved intent-to-ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/MkSfvCan7kY/PgniO8jw...


BUG=624361
==========

to

==========
Change the linebreak behavior of minlength and maxlength attributes of TEXTAREA.

Change it in response to a specification change, and to improve compatibility
with
Firefox and Edge.

Old behavior:
 CRLF, CR, or LF is counted as 2 because we limited the submission value.

New behavior:
 CRLF, CR, or LF is counted as 1 because we limit the API value.

We can use HTMLTextAreaElement::value().length() as is because it's the API
value,
and other string input should be counted by computeLengthForAPIValue().

Approved intent-to-ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/MkSfvCan7kY/PgniO8jw...

BUG=624361

Committed: https://crrev.com/a15474ff7adff41a8956ab436ac63a7d5b7a090f
Cr-Commit-Position: refs/heads/master@{#421445}
==========

[commit-bot: I haz the power, 2016-09-28 04:24:05]

Patchset 1 (id:??) landed as
https://crrev.com/a15474ff7adff41a8956ab436ac63a7d5b7a090f
Cr-Commit-Position: refs/heads/master@{#421445}