HTMLOptionElement.prototype.selected setter should not make an option dirty if it doesn't change it…

HTMLOptionElement.prototype.selected setter should not make an option dirty if it doesn't change its selectedness and the option is owned by a select element.

We implmeneted option dirtiness in [1] and it conformed to the standard.
However, it was not compatible with other browsers and caused a regression.
This CL makes the dirtiness behavior compatible with Firefox.

[1] crrev.com/363172

BUG=570367
TEST=automated

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

[tkent, 2015-12-21 07:39:28]

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

[tkent, 2015-12-21 07:39:29]

Keishi, would you review this please?

[keishi, 2015-12-23 12:08:49]

LGTM

[tkent, 2015-12-23 23:00:57]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2015-12-23 23:02:18]

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

[tfarina, 2015-12-23 23:04:37]

https://codereview.chromium.org/1541753002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/HTMLOptionElement.cpp (right):

https://codereview.chromium.org/1541753002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLOptionElement.cpp:273: // As of December
2015, the HTML specification says the dirtiness becomes
is it worth pointing to the link for that specification?

[commit-bot: I haz the power, 2015-12-24 00:28:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-24 00:29:52]

Description was changed from

==========
HTMLOptionElement.prototype.selected setter should not make an option dirty if
it doesn't change its selectedness and the option is owned by a select element.

We implmeneted option dirtiness in [1] and it conformed to the standard.
However, it was not compatible with other browsers and caused a regression.
This CL makes the dirtiness behavior compatible with Firefox.

[1] crrev.com/363172

BUG=570367
TEST=automated
==========

to

==========
HTMLOptionElement.prototype.selected setter should not make an option dirty if
it doesn't change its selectedness and the option is owned by a select element.

We implmeneted option dirtiness in [1] and it conformed to the standard.
However, it was not compatible with other browsers and caused a regression.
This CL makes the dirtiness behavior compatible with Firefox.

[1] crrev.com/363172

BUG=570367
TEST=automated

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

[commit-bot: I haz the power, 2015-12-24 00:29:53]

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