Implement HTMLOptionElement.form as required by spec.

Implement HTMLOptionElement.form as required by spec.

The spec tells us that an option element

  http://www.whatwg.org/specs/web-apps/current-work//#dom-option-form

only has a form value if inside a <select>. Implement this.

R=arv@chromium.org
BUG=324828
TEST=fast/forms/form-attribute

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=163671

[sof, 2013-12-10 11:24:22]

At your leisure, please take a look.

[CL 2 (of 3) from https://codereview.chromium.org/107183004/ ]

[sof, 2013-12-10 13:03:04]

On 2013/12/10 11:24:22, sof wrote:
> At your leisure, please take a look.
> 
> [CL 2 (of 3) from https://codereview.chromium.org/107183004/ ]

Final (&main) part cannot go ahead without this one, so will wait.

[arv (Not doing code reviews), 2013-12-10 20:28:44]

LGTM

[commit-bot: I haz the power, 2013-12-10 21:05:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sigbjornf@opera.com/102263006/1

[tkent, 2013-12-10 23:09:07]

lgtm

https://codereview.chromium.org/102263006/diff/1/Source/core/html/HTMLOptionE...
File Source/core/html/HTMLOptionElement.cpp (right):

https://codereview.chromium.org/102263006/diff/1/Source/core/html/HTMLOptionE...
Source/core/html/HTMLOptionElement.cpp:371: HTMLSelectElement* selectElement =
ownerSelectElement();
nit:

if (HTMLSelectElement* selectElement = ownerSelectElement())
    return selectElement->formOwner();
return 0;

is simpler.

[commit-bot: I haz the power, 2013-12-10 23:22:47]

Commit queue rejected this change because the description was changed
between the time the change entered the commit queue and the time it
was ready to commit. You can safely check the commit box again.

[commit-bot: I haz the power, 2013-12-11 05:18:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sigbjornf@opera.com/102263006/1

[commit-bot: I haz the power, 2013-12-11 05:19:13]

Change committed as 163671