Fix problems on changing 'multiple' state SELECT element to 'single' state.

Fix problems on changing 'multiple' state SELECT element to 'single' state.

This CL fixes two problems.

* Setting 'multiple' IDL attribute to false and removing 'multiple' content
  attribute had different behavior.
  They should be identical.  This CL removes HTMLSelectElement::setMultiple(), and
  add [Reflect] to 'multiple' IDL attribute.

* If multiple SELECT without selected OPTIONs is changed to single, we should run
  resetToDefaultSelection().
  All other browsers agree with this behavior.

BUG=599008
Committed: https://crrev.com/ed6cd6b785c97d4a678c262bbf5e62bf46e5a2e8
Cr-Commit-Position: refs/heads/master@{#384226}

[tkent, 2016-03-30 13:07:56]

Description was changed from

==========
Changing 'multiple' state of SELECT element should reset to default selection.

BUG=599008
==========

to

==========
Fix problems on changing 'multiple' state SELECT element to 'single' state.

This CL fixes two problems.

* Setting 'multiple' IDL attribute to false and removing 'multiple' content
  attribute has different behavior.
  They should be identical.  This CL removes HTMLSelectElement::setMultiple(),
and
  add [Reflect] to 'multiple' IDL attribute.

* If multiple SELECT without selected OPTIONs is changed to single, we should
run
  resetToDefaultSelection().
  All other browsers agree with this behavior.


BUG=599008
==========

[tkent, 2016-03-30 13:08:16]

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

[tkent, 2016-03-30 13:08:16]

keishi, would you review this please?

[tkent, 2016-03-30 13:08:34]

Description was changed from

==========
Fix problems on changing 'multiple' state SELECT element to 'single' state.

This CL fixes two problems.

* Setting 'multiple' IDL attribute to false and removing 'multiple' content
  attribute has different behavior.
  They should be identical.  This CL removes HTMLSelectElement::setMultiple(),
and
  add [Reflect] to 'multiple' IDL attribute.

* If multiple SELECT without selected OPTIONs is changed to single, we should
run
  resetToDefaultSelection().
  All other browsers agree with this behavior.


BUG=599008
==========

to

==========
Fix problems on changing 'multiple' state SELECT element to 'single' state.

This CL fixes two problems.

* Setting 'multiple' IDL attribute to false and removing 'multiple' content
  attribute had different behavior.
  They should be identical.  This CL removes HTMLSelectElement::setMultiple(),
and
  add [Reflect] to 'multiple' IDL attribute.

* If multiple SELECT without selected OPTIONs is changed to single, we should
run
  resetToDefaultSelection().
  All other browsers agree with this behavior.


BUG=599008
==========

[keishi, 2016-03-31 02:23:56]

LGTM

[tkent, 2016-03-31 02:29:14]

The CQ bit was checked by tkent@chromium.org

[tkent, 2016-03-31 02:29:14]

The patchset sent to the CQ was uploaded after l-g-t-m from keishi@chromium.org
Link to the patchset: https://codereview.chromium.org/1838183003/#ps40001
(title: "Fix a typo in a comment")

[commit-bot: I haz the power, 2016-03-31 02:29:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1838183003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1838183003/40001

[tkent, 2016-03-31 06:11:15]

The CQ bit was unchecked by tkent@chromium.org

[tkent, 2016-03-31 06:13:34]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-03-31 06:13:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1838183003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1838183003/40001

[tkent, 2016-03-31 07:57:41]

The CQ bit was unchecked by tkent@chromium.org

[tkent, 2016-03-31 08:16:09]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-03-31 08:16:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1838183003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1838183003/40001

[commit-bot: I haz the power, 2016-03-31 08:30:21]

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

[commit-bot: I haz the power, 2016-03-31 08:30:22]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_TIMED_OUT,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[tkent, 2016-03-31 09:01:32]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-03-31 09:02:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1838183003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1838183003/40001

[commit-bot: I haz the power, 2016-03-31 09:07:03]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-31 09:08:03]

Description was changed from

==========
Fix problems on changing 'multiple' state SELECT element to 'single' state.

This CL fixes two problems.

* Setting 'multiple' IDL attribute to false and removing 'multiple' content
  attribute had different behavior.
  They should be identical.  This CL removes HTMLSelectElement::setMultiple(),
and
  add [Reflect] to 'multiple' IDL attribute.

* If multiple SELECT without selected OPTIONs is changed to single, we should
run
  resetToDefaultSelection().
  All other browsers agree with this behavior.


BUG=599008
==========

to

==========
Fix problems on changing 'multiple' state SELECT element to 'single' state.

This CL fixes two problems.

* Setting 'multiple' IDL attribute to false and removing 'multiple' content
  attribute had different behavior.
  They should be identical.  This CL removes HTMLSelectElement::setMultiple(),
and
  add [Reflect] to 'multiple' IDL attribute.

* If multiple SELECT without selected OPTIONs is changed to single, we should
run
  resetToDefaultSelection().
  All other browsers agree with this behavior.

BUG=599008

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

[commit-bot: I haz the power, 2016-03-31 09:08:05]

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