[patch][ccla] the gfx surface is not clipped under IE9/svg

[Patrick Ruzand]

When a page is rendered in standard mode, the svg surface does not clip under IE9, as opposed to the Chrome/FF. In fact, IE9 is (to my understanding) correct wrt the SVG spec:

from: ​http://www.w3.org/TR/SVG/masking.html#OverflowProperty

"The initial value for ‘overflow’ as defined in [CSS2-overflow] is 'visible', and this applies also to the root ‘svg’ element;"

Fix: explicitely set the surface rawNode overflow attribute to 'hidden' when the surface is created.

[Patrick Ruzand]

patch + test case

[Chris Mitchell]

(In [25216]) fixes #13092 initial overflow for svg should default to hidden on Surface to fix clipping bug on ie9 \!strict

[Chris Mitchell]

(In [25217]) fixes #13092 initial overflow for svg should default to hidden on Surface to fix clipping bug on ie9 \!strict

[Chris Mitchell]

(In [25218]) refs #13092 updated testcase text for surface clipping\!strict

[Eugene Lazutkin]

Heh, it looks like both chrism and I tried to commit the patch: [25219].

I had to reformat the patch:

1. Please follow the style guide.
2. Please use "" quotes consistently, use only if you have to. And try not to mix both, if it is possible. I bet it was possible in this case.
3. Don't do multiple spaces in a row unless you want to align something to improve readability.
4. Make sure that text corresponds to code --- it was "Testing arc", so I was looking for it, instead it was a clipped rectangle.
5. Even in HTML try to show the structure.

Other than that it was an excellent patch: small, practical, low-risk.

[Eugene Lazutkin]

In the point 2 above it should be double quotes vs. single quotes. The latter got interpreted as "switch to italic".

[Eugene Lazutkin]

Yeah, forgot to say that when we glue words together (in identifiers, or file names) we usually use a camel case nowadays.

In general I found that mimicking a surrounding code is a good rule of thumb --- improves readability. Otherwise it'll look like a patchwork. So if in doubt --- look around.

[Kenneth G. Franqueiro]

Updating milestone to 1.5.2 to reflect inclusion in changeset [26956] for ticket #14199.