Fix hitting assert when opening a select menu with no options

Fix hitting assert when opening a select menu with no options

TEST=Layout test be added in https://codereview.chromium.org/736883002/
BUG=346582
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=187124

[keishi, 2014-12-15 08:32:14]

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

[tkent, 2014-12-15 08:56:25]

lgtm

https://codereview.chromium.org/799343002/diff/1/Source/core/html/HTMLSelectE...
File Source/core/html/HTMLSelectElement.cpp (left):

https://codereview.chromium.org/799343002/diff/1/Source/core/html/HTMLSelectE...
Source/core/html/HTMLSelectElement.cpp:1211: setSelectedIndex(index);
Hmm, setSelectedIndex(selectedIndex()) looks to do nothing, and safe to remove.

[keishi, 2014-12-15 09:01:22]

https://codereview.chromium.org/799343002/diff/1/Source/core/html/HTMLSelectE...
File Source/core/html/HTMLSelectElement.cpp (left):

https://codereview.chromium.org/799343002/diff/1/Source/core/html/HTMLSelectE...
Source/core/html/HTMLSelectElement.cpp:1211: setSelectedIndex(index);
On 2014/12/15 08:56:25, unavailable until Feburuary wrote:
> Hmm, setSelectedIndex(selectedIndex()) looks to do nothing, and safe to
remove.

Yup

[keishi, 2014-12-15 09:01:25]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2014-12-15 09:01:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/799343002/1

[commit-bot: I haz the power, 2014-12-15 10:27:18]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=187124