/[blink]
Chromium logo

Revision 174338


Jump to revision: Previous Next
Author: kouhei@chromium.org
Date: Tue May 20 02:13:08 2014 UTC (9 years ago)
Changed paths: 6
Log Message:
SVG: Moving animating <svg> to other iframe should not crash.

Moving SVGSVGElement with its SMILTimeContainer already started caused crash before this patch.
|SVGDocumentExtentions::startAnimations()| calls begin() against all SMILTimeContainers in the document, but the SMILTimeContainer for <svg> moved from other document may be already started.

BUG=369860

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

Changed paths

Path Details
Directorytrunk/LayoutTests/svg/dom/move-animating-svg-to-other-frame-expected.txt
(Copied from trunk/LayoutTests/accessibility/insert-selected-option-into-select-causes-crash-expected.txt, r174337)
added , text changed , props changed
Directorytrunk/LayoutTests/svg/dom/move-animating-svg-to-other-frame.html added
Directorytrunk/LayoutTests/svg/dom/resources/import-other-svg.svg added
Directorytrunk/LayoutTests/svg/dom/resources/svg-with-animate.svg added
Directorytrunk/Source/core/svg/SVGDocumentExtensions.cpp modified , text changed
Directorytrunk/Source/core/svg/animation/SMILTimeContainer.cpp modified , text changed

Properties

Name Value
commit-bot commit-bot@chromium.org

Powered by ViewVC 1.1.26 ViewVC Help