SELECT: Store the suggested value by an HTMLOptionElement pointer instead of an index number.

SELECT: Store the suggested value by an HTMLOptionElement pointer instead of an index number.

Managing an OPTION by an index number is a bad idea in general because an index
number is fragile.  This CL changes m_suggestedIndex from |int| to
|Member<HTMLOptionElement>|, and fix a bug about OPTION addition and removal.

Committed: https://crrev.com/cf0c390214da2eeaa9407597793fb8f2ff2fda56
Cr-Commit-Position: refs/heads/master@{#403860}

[tkent, 2016-07-06 04:13:10]

Patchset #1 (id:1) has been deleted

[tkent, 2016-07-06 04:58:03]

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

[tkent, 2016-07-06 04:58:04]

Keishi, would you review this please?

[keishi, 2016-07-06 05:53:45]

LGTM

[tkent, 2016-07-06 05:59:21]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-07-06 05:59:25]

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-07-06 07:12:08]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-07-06 07:13:43]

Description was changed from

==========
SELECT: Store the suggested value by an HTMLOptionElement pointer instead of an
index number.

Managing an OPTION by an index number is a bad idea in general because an index
number is fragile.  This CL changes m_suggestedIndex from |int| to
|Member<HTMLOptionElement>|, and fix a bug about OPTION addition and removal.
==========

to

==========
SELECT: Store the suggested value by an HTMLOptionElement pointer instead of an
index number.

Managing an OPTION by an index number is a bad idea in general because an index
number is fragile.  This CL changes m_suggestedIndex from |int| to
|Member<HTMLOptionElement>|, and fix a bug about OPTION addition and removal.

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

[commit-bot: I haz the power, 2016-07-06 07:13:44]

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