Node::lazyAttach doesn't need to mark all children as needing a FullStyleChange
Node::lazyAttach doesn't need to mark all children as needing a FullStyleChange

Marking yourself as needing a FullStyleChange will cause a Force style recalc to go
down through all your descendants. This means we don't need to set every descendant
of the lazyAttach node with FullStyleChange or manually set the childNeedsStyleRecalc
bits.

This should have no behavior change except that HTMLFrameElementBase::didNotifySubtreeInsertions
will no longer walk through all the descendants of the <iframe>. In practice no one
appends anything inside <iframe> elements so it's unlikely this is a performance
improvement.

This also makes it easier to test the behavior from https://codereview.chromium.org/15159008
where lazyAttach doesn't set the attached bit by making the default value for the
shouldSetAttached argument to lazyAttach be DoNotSetAttached.

R=ojan@chromium.org

Review URL: https://codereview.chromium.org/16819004

git-svn-id: bf5cd6ccde378db821296732a091cfbcf5285fbd@152467 bbb929c8-8fbe-4397-9dbb-9b2b20218538

axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk,axinging/chromium-crosswalk