Add Element::subResourceAttributeName() virtual function

Add Element::subResourceAttributeName() virtual function

Add Element::subResourceAttributeName() virtual function to distribute the code
to each HTML class, instead of having a complex function with a lot of branching
to determine the subresource attribute name of an Element.

R=tkent, adamk

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

[tkent, 2014-03-19 01:00:44]

lgtm

https://codereview.chromium.org/202633009/diff/1/Source/core/dom/Element.cpp
File Source/core/dom/Element.cpp (right):

https://codereview.chromium.org/202633009/diff/1/Source/core/dom/Element.cpp#...
Source/core/dom/Element.cpp:1076: return 0;
nit: We may return WebCore::nullQName(), or we may change this function to
subResourceAttributeValue().

[Inactive, 2014-03-19 13:51:42]

https://codereview.chromium.org/202633009/diff/1/Source/core/dom/Element.cpp
File Source/core/dom/Element.cpp (right):

https://codereview.chromium.org/202633009/diff/1/Source/core/dom/Element.cpp#...
Source/core/dom/Element.cpp:1076: return 0;
On 2014/03/19 01:00:44, tkent wrote:
> nit: We may return WebCore::nullQName(), or we may change this function to
> subResourceAttributeValue().

Done, returning nullQName() now, one less pointer, yay.

[Inactive, 2014-03-19 13:52:01]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-03-19 13:52:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/202633009/20001

[commit-bot: I haz the power, 2014-03-19 15:58:34]

Change committed as 169558