Add use counter for hasAttributes

Add use counter for hasAttributes

hasAttributes was removed from DOM4.

BUG=

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

[ojan, 2013-09-07 17:02:02]

lgtm


On Fri, Sep 6, 2013 at 8:36 AM, <arv@chromium.org> wrote:

> Reviewers: Christophe Dumez, ojan,
>
> Description:
> Add use counter for hasAttributes
>
> hasAttributes was removed from DOM4.
>
> BUG=
>
> Please review this at
https://codereview.chromium.**org/23561008/<https://codereview.chromium.org/2...
>
> SVN Base:
svn://svn.chromium.org/blink/**trunk<http://svn.chromium.org/blink/trunk>
>
> Affected files (+2, -1 lines):
>   M Source/core/dom/Element.idl
>   M Source/core/page/UseCounter.h
>
>
> Index: Source/core/dom/Element.idl
> diff --git a/Source/core/dom/Element.idl b/Source/core/dom/Element.idl
> index 4a8cb72bfcefbab1d37479b99b1f04**1294c32e6b..**
> 3de21737d5b41b68e24404ce87629d**7f1b50da3d 100644
> --- a/Source/core/dom/Element.idl
> +++ b/Source/core/dom/Element.idl
> @@ -36,7 +36,7 @@
>      [PerWorldBindings] NodeList getElementsByTagName([Default=**Undefined]
> optional DOMString name);
>
>      [PerWorldBindings] readonly attribute NamedNodeMap     attributes;
> -    boolean            hasAttributes();
> +    [MeasureAs=HasAttributes] boolean hasAttributes();
>
>      // DOM Level 2 Core
>
> Index: Source/core/page/UseCounter.h
> diff --git a/Source/core/page/UseCounter.**h
> b/Source/core/page/UseCounter.**h
> index f70e966cece805512d309f6fff51ae**1ff07e163d..**
> b715554d7dc3f0c2ce20c0f3a0a03d**be2e637cfa 100644
> --- a/Source/core/page/UseCounter.**h
> +++ b/Source/core/page/UseCounter.**h
> @@ -197,6 +197,7 @@ public:
>          UnprefixedAnimationIterationEv**ent,
>          PrefixedAndUnprefixedAnimation**IterationEvent,
>          EventReturnValue, // Legacy IE extension.
> +        HasAttributes, // Removed from DOM4.
>          // Add new features immediately above this line. Don't change
> assigned
>          // numbers of each items, and don't reuse unused slots.
>          NumberOfFeatures, // This enum value must be last.
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[do-not-use, 2013-09-07 17:10:42]

lgtm

[commit-bot: I haz the power, 2013-09-09 16:27:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/arv@chromium.org/23561008/5001

[commit-bot: I haz the power, 2013-09-09 17:32:00]

Change committed as 157461