SELECT element: Remove HTMLSelectElement::optionToListIndex().

SELECT element: Remove HTMLSelectElement::optionToListIndex().

Remove two instances of optionToListIndex() callsites, and remove
optionToListIndex().

* HTMLSelectElement::hasPlaceholderLabelOption():
  optionToListIndex() was just redundant. We need to check just
  isHTMLOptionElement(listItems()[0]).

* HTMLSelectElement::remove():
  We can use item() to get an OPTION at the specified index.

This CL has no behavior changes.

Committed: https://crrev.com/8f6281da70545e44a3445840e05a24ae79498e8f
Cr-Commit-Position: refs/heads/master@{#405690}

[tkent, 2016-07-15 02:04:22]

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

[commit-bot: I haz the power, 2016-07-15 02:04:41]

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

[tkent, 2016-07-15 02:24:54]

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

[tkent, 2016-07-15 02:24:54]

Keishi, would you review this please?

[tkent, 2016-07-15 02:25:28]

Description was changed from

==========
SELECT element: Remove HTMLSelectElement::optionToListIndex().

Remove two instances of optionToListIndex() callsites, and remove
optionToListIndex().

* HTMLSelectElement::hasPlaceholderLabelOption():
  optionToListIndex() was just redandunt. We need to check just
  isHTMLOptionElement(listItems()[0]).

* HTMLSelectElement::remove():
  We can use item() to get an OPTION at the specified index.

This CL has no behavior changes.
==========

to

==========
SELECT element: Remove HTMLSelectElement::optionToListIndex().

Remove two instances of optionToListIndex() callsites, and remove
optionToListIndex().

* HTMLSelectElement::hasPlaceholderLabelOption():
  optionToListIndex() was just redundant. We need to check just
  isHTMLOptionElement(listItems()[0]).

* HTMLSelectElement::remove():
  We can use item() to get an OPTION at the specified index.

This CL has no behavior changes.
==========

[keishi, 2016-07-15 02:27:21]

LGTM

[commit-bot: I haz the power, 2016-07-15 03:20:05]

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

[commit-bot: I haz the power, 2016-07-15 03:20:05]

Dry run: This issue passed the CQ dry run.

[tkent, 2016-07-15 03:29:17]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-07-15 03:29:28]

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-15 03:33:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-15 03:33:50]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-15 03:35:08]

Description was changed from

==========
SELECT element: Remove HTMLSelectElement::optionToListIndex().

Remove two instances of optionToListIndex() callsites, and remove
optionToListIndex().

* HTMLSelectElement::hasPlaceholderLabelOption():
  optionToListIndex() was just redundant. We need to check just
  isHTMLOptionElement(listItems()[0]).

* HTMLSelectElement::remove():
  We can use item() to get an OPTION at the specified index.

This CL has no behavior changes.
==========

to

==========
SELECT element: Remove HTMLSelectElement::optionToListIndex().

Remove two instances of optionToListIndex() callsites, and remove
optionToListIndex().

* HTMLSelectElement::hasPlaceholderLabelOption():
  optionToListIndex() was just redundant. We need to check just
  isHTMLOptionElement(listItems()[0]).

* HTMLSelectElement::remove():
  We can use item() to get an OPTION at the specified index.

This CL has no behavior changes.

Committed: https://crrev.com/8f6281da70545e44a3445840e05a24ae79498e8f
Cr-Commit-Position: refs/heads/master@{#405690}
==========

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

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