Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element
by using parseHTMLInteger instead of toInt.

This also fixes the last remaining
reflection-grouping.html failures.

BUG=651762, 634796
Committed: https://crrev.com/d119af9ade51292737ff2c2730f5a14abed0004b
Cr-Commit-Position: refs/heads/master@{#431495}

[rwlbuis, 2016-11-10 13:49:47]

Description was changed from

==========
LI WIP

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

to

==========
Fix reflection-grouping.html

Fix reflection-grouping.html by fixing the last
remaining problem, in HTMLOListElement parse the start
attribute using parseHTMLInteger instead of toInt.

BUG=651762
==========

[rwlbuis, 2016-11-10 13:50:42]

rob.buis@samsung.com changed reviewers:
+ tkent@chromium.org

[rwlbuis, 2016-11-10 13:50:42]

PTAL.

[rwlbuis, 2016-11-10 18:19:13]

Description was changed from

==========
Fix reflection-grouping.html

Fix reflection-grouping.html by fixing the last
remaining problem, in HTMLOListElement parse the start
attribute using parseHTMLInteger instead of toInt.

BUG=651762
==========

to

==========
Fix reflection-grouping.html

Fix reflection-grouping.html by fixing the last
remaining problem, in HTMLOListElement parse the start
attribute using parseHTMLInteger instead of toInt.

BUG=651762, 634796
==========

[tkent, 2016-11-10 22:59:20]

lgtm.

IMO, the CL summary should be something like "Fix parsing of 'start' attribute
of <ol> element" rather than "Fix reflection-grouping.html"

[rwlbuis, 2016-11-10 23:41:11]

On 2016/11/10 22:59:20, tkent wrote:
> lgtm.
> 
> IMO, the CL summary should be something like "Fix parsing of 'start' attribute
> of <ol> element" rather than "Fix reflection-grouping.html"

Good point, will fix.

[rwlbuis, 2016-11-10 23:45:36]

Description was changed from

==========
Fix reflection-grouping.html

Fix reflection-grouping.html by fixing the last
remaining problem, in HTMLOListElement parse the start
attribute using parseHTMLInteger instead of toInt.

BUG=651762, 634796
==========

to

==========
Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element
by using parseHTMLInteger instead of toInt.

This also fixes the last remaining reflection-grouping.html
failures.

BUG=651762, 634796
==========

[rwlbuis, 2016-11-10 23:46:26]

Description was changed from

==========
Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element
by using parseHTMLInteger instead of toInt.

This also fixes the last remaining reflection-grouping.html
failures.

BUG=651762, 634796
==========

to

==========
Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element
by using parseHTMLInteger instead of toInt.

This also fixes the last remaining
reflection-grouping.html failures.

BUG=651762, 634796
==========

[rwlbuis, 2016-11-10 23:46:49]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2016-11-10 23:47:33]

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-11-11 00:00:03]

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

[commit-bot: I haz the power, 2016-11-11 00:00:04]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[rwlbuis, 2016-11-11 01:58:39]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2016-11-11 01:59:18]

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-11-11 04:26:28]

Description was changed from

==========
Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element
by using parseHTMLInteger instead of toInt.

This also fixes the last remaining
reflection-grouping.html failures.

BUG=651762, 634796
==========

to

==========
Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element
by using parseHTMLInteger instead of toInt.

This also fixes the last remaining
reflection-grouping.html failures.

BUG=651762, 634796
==========

[commit-bot: I haz the power, 2016-11-11 04:26:29]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-11 05:16:16]

Description was changed from

==========
Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element
by using parseHTMLInteger instead of toInt.

This also fixes the last remaining
reflection-grouping.html failures.

BUG=651762, 634796
==========

to

==========
Fix parsing of 'start' attribute of <ol> element

Fix parsing of 'start' attribute of <ol> element
by using parseHTMLInteger instead of toInt.

This also fixes the last remaining
reflection-grouping.html failures.

BUG=651762, 634796

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

[commit-bot: I haz the power, 2016-11-11 05:16:17]

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