Invalidate SVG filter results in SVGResourcesCache::clientLayoutChanged().

Invalidate SVG filter results in SVGResourcesCache::clientLayoutChanged().

This is a regression introduced by http://trac.webkit.org/changeset/103539.

For the SVG tree, filter result invalidation is handled by
RenderSVGResource::markForLayoutAndParentResourceInvalidation. But non-SVG content (such as
a <foreignObject> HTML subtree) does not call markForLayoutAndParentResourceInvalidation and
instead uses the regular setNeedsLayout() invalidation path.

Hence, changes in the HTML subtree do not invalidate cached results for filters applied on
ancestor SVG elements. In order to cover this case, clientLayoutChanged() needs to invalidate
filter results even if the SVG element itself does not need a re-layout. Note that prior to
http://trac.webkit.org/changeset/103539 the method used to do exactly that, but the branch was
removed on the assumption that markForLayoutAndParentResourceInvalidation() is already taking care
of it. Obviously, the assumption doesn't hold for non-SVG content.

BUG=165180
R=dschulze@chromium.org,schenney@chromium.org,pdr@chromium.org

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

[f(malita), 2013-05-17 22:00:32]

patch.UnsupportedPatchFormat: Can't process patch for file
LayoutTests/svg/foreignObject/filter-repaint-expected.svg.
Binary file support is temporarilly disabled due to a bug. Please commit blindly
the binary files first then commit the source change as a separate CL. Sorry for
the annoyance.

Huh? The bots have been smoking some dubious stuff all day, but this is
something else...

[Stephen Chennney, 2013-05-17 22:09:02]

On 2013/05/17 22:00:32, Florin Malita wrote:
> patch.UnsupportedPatchFormat: Can't process patch for file
> LayoutTests/svg/foreignObject/filter-repaint-expected.svg.
> Binary file support is temporarilly disabled due to a bug. Please commit
blindly
> the binary files first then commit the source change as a separate CL. Sorry
for
> the annoyance.
> 
> Huh? The bots have been smoking some dubious stuff all day, but this is
> something else...

Someone incorrectly considers svg as binary? Or maybe it was left off some
whitelist. Have you asked on blink-dev or IRC about any changes to the try bots?

[Stephen Chennney, 2013-05-17 22:09:23]

lgtm

[f(malita), 2013-05-20 14:29:56]

On 2013/05/17 22:09:02, Stephen Chenney wrote:
> On 2013/05/17 22:00:32, Florin Malita wrote:
> > patch.UnsupportedPatchFormat: Can't process patch for file
> > LayoutTests/svg/foreignObject/filter-repaint-expected.svg.
> > Binary file support is temporarilly disabled due to a bug. Please commit
> blindly
> > the binary files first then commit the source change as a separate CL. Sorry
> for
> > the annoyance.
> > 
> > Huh? The bots have been smoking some dubious stuff all day, but this is
> > something else...
> 
> Someone incorrectly considers svg as binary? Or maybe it was left off some
> whitelist. Have you asked on blink-dev or IRC about any changes to the try
bots?

It's a known issue, now tracked here:
https://code.google.com/p/chromium/issues/detail?id=242042

Since we can't land SVG files cleanly, I'll split this up and land the
suppressed test first, then the code patch + suppression removal.

[f(malita), 2013-05-20 15:11:40]

Suppressed test landed: 
https://src.chromium.org/viewvc/blink?revision=150671&view=revision

[commit-bot: I haz the power, 2013-05-20 15:13:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fmalita@chromium.org/15341004/10001

[commit-bot: I haz the power, 2013-05-20 15:50:04]

Change committed as 150675