Sync HTML element interfaces O-Q with the spec

Sync HTML element interfaces O-Q with the spec

Also fix the confused comment in HTMLImageElement, its hspace/vspace
attributes should also be unsigned long, noticed now because of the same
attributes on HTMLObjectElement.

There are no changes to the generated code other than order.

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

[philipj_slow, 2015-03-21 18:33:42]

philipj@opera.com changed reviewers:
+ jl@opera.com

[philipj_slow, 2015-03-21 18:33:42]

PTAL

[philipj_slow, 2015-03-21 18:36:21]

oops

[Jens Widell, 2015-03-21 18:38:38]

LGTM

[philipj_slow, 2015-03-21 18:44:20]

and P*, Q*

[philipj_slow, 2015-03-21 18:45:44]

Uh sorry, I didn't expect to get review so fast so I threw in some more. Still
good?

[vivekg, 2015-03-22 02:27:02]

vivekg@chromium.org changed reviewers:
+ vivekg@chromium.org

[vivekg, 2015-03-22 02:27:03]

lgtm

https://codereview.chromium.org/1028633005/diff/40001/Source/core/html/HTMLIm...
File Source/core/html/HTMLImageElement.idl (right):

https://codereview.chromium.org/1028633005/diff/40001/Source/core/html/HTMLIm...
Source/core/html/HTMLImageElement.idl:23: // FIXME: Most long types in this
interfaces should be unsigned long.
Which ones need to be 'unsigned long'? Your TODO talks about hspace and vspace.
Can we combine these two things?

[philipj_slow, 2015-03-22 04:41:07]

clear up HTMLImageElement confusion

[philipj_slow, 2015-03-22 04:43:40]

On 2015/03/22 02:27:03, vivekg_ wrote:
> lgtm
> 
>
https://codereview.chromium.org/1028633005/diff/40001/Source/core/html/HTMLIm...
> File Source/core/html/HTMLImageElement.idl (right):
> 
>
https://codereview.chromium.org/1028633005/diff/40001/Source/core/html/HTMLIm...
> Source/core/html/HTMLImageElement.idl:23: // FIXME: Most long types in this
> interfaces should be unsigned long.
> Which ones need to be 'unsigned long'? Your TODO talks about hspace and
vspace.
> Can we combine these two things?

Thanks for paying attention, I was confused/wrong while dealing with
HTMLImageElement earlier. Fixed now.

[philipj_slow, 2015-03-22 04:43:56]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-03-22 04:43:57]

The patchset sent to the CQ was uploaded after l-g-t-m from jl@opera.com,
vivekg@chromium.org
Link to the patchset: https://codereview.chromium.org/1028633005/#ps60001
(title: "clear up HTMLImageElement confusion")

[commit-bot: I haz the power, 2015-03-22 04:44:14]

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

[commit-bot: I haz the power, 2015-03-22 06:28:25]

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

[commit-bot: I haz the power, 2015-03-22 06:28:27]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/5...)

[philipj_slow, 2015-03-22 09:55:06]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-03-22 09:55:13]

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

[commit-bot: I haz the power, 2015-03-22 11:31:56]

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

[commit-bot: I haz the power, 2015-03-22 11:31:59]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/5...)

[philipj_slow, 2015-03-22 15:23:40]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-03-22 15:23:47]

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

[commit-bot: I haz the power, 2015-03-22 17:10:41]

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