Avoid calling Node::numberOfScopedHTMLStyleChildren() to check for existence

Avoid calling Node::numberOfScopedHTMLStyleChildren() to check for existence

Avoid calling Node::numberOfScopedHTMLStyleChildren() to check for existence as
we don't want to traverse all children merely to check for the existence
of 1 child. Node::hasScopedHTMLStyleChild() can be used to check this instead
as we have a dedicated flag for this. Also, don't unnecessarily traverse all
children to set the HasScopedHTMLStyleChildFlag, and break early instead as
soon as one child is found.

R=adamk, eseidel

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

[esprehn, 2014-02-24 19:43:43]

lgtm, but why do we need numberOfScopedHTMLStyleChildren at all?

[Inactive, 2014-02-24 20:13:44]

On 2014/02/24 19:43:43, esprehn wrote:
> lgtm, but why do we need numberOfScopedHTMLStyleChildren at all?

It is currently exposed to layout test via Internals API.

[Inactive, 2014-02-24 20:19:15]

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

[commit-bot: I haz the power, 2014-02-24 20:19:25]

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

[commit-bot: I haz the power, 2014-02-25 01:05:09]

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

[commit-bot: I haz the power, 2014-02-25 05:16:53]

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

[commit-bot: I haz the power, 2014-02-25 06:36:25]

Change committed as 167728