CSS shapes editor crashes in Chrome when deleting the parameters in a shape function.

[RaymondLim]

1. Open https://gist.github.com/RaymondLim/9987650 file in Brackets.
2. Live preview in Chrome.
3. Set cursor before ) in ellipse function on line 9. You will see shapes editor in Chrome browser.
4. Hit backspace in Brackets to delete the entire parameter.

Result: Chrome shows a crash message "Aw, Snap! Something went wrong showing this webpage."

[RaymondLim]

@oslego Not sure your new pr covers this or not.

[oslego]

@RaymondLim see the end of my comment for #7424

circle({something} at) and ellipse({something}*2 at) is a bug in the browser implementation, which causes a crash. It happens regardless of the extension. I don't think there's a way to guard against it.

[oslego]

The issue has been fixed, but it will take time to trickle down into Chrome Beta then Stable.

[JeffryBooher]

I reproduced this when quitting brackets while the shape editor is active but i just get the He's dead Jim page which

[RaymondLim]

Chrome bug

[oslego]

Yes, circle(r at cx cy) is the new CSS Shapes syntax as of Chrome 34. It used to be circle(cx, cy, r) prior to that release. Dual shape notation may work in the browser during the transition, but the shapes editor extension will only work with the new syntax. The old is deemed invalid and ignored.

The browser crash for incomplete circle(r at) or ellipse(rx ry at) occurs because of a browser implementation bug of CSS Shapes.

WebKit: https://bugs.webkit.org/show_bug.cgi?id=130366
Blink: https://code.google.com/p/chromium/issues/detail?id=353774

The bug has been fixed in Nightly Webkit and Blink, but for some time it will affect Chrome Stable and Beta.

[pthiess]

@RaymondLim to track.