[SVG] Interval should not be created if a value in begin-value-list doesn't have a matching value i…

[SVG] Interval should not be created if a value in begin-value-list doesn't have a matching value in end-value-list

In the pseudo code in http://www.w3.org/TR/SMIL3/smil-timing.html#q90 ,
it says that if all ends are before the current begin, then it is a bad interval and current begin will be discarded.
Existing code is treating it as a vaild interval with end as indefinite.
Modified the logic to be in line with the pseudo code.

R=pdr@chromium.org, schenney@chromium.org, dschulze@chromium.org, fmalita@chromium.org
BUG=269059
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=157739

[pavane, 2013-08-08 19:09:11]

WIP patch.

Please have a look at this patch.

I am having nightmares writing layout test cases for svg animations :) Please
help!

For your reference, please have a look at below three patches:
https://codereview.chromium.org/22604002
https://codereview.chromium.org/22706003
https://codereview.chromium.org/22385010

For first and second, i wrote layout tests using js test suite for svg
animations and for third one, i used a ref test.

If you test these patches manually, the scenarios for which i made these patches
work well but  when i write layout test cases for those scenarios, they fail.

In the first patch, layout test case hangs whenever i query position at a time
stamp after animation end. Same can be reproduced with a ref test using
setCurrentTime(t) for t >= end

In second and third patches also, the setCurrentTime() method doesn't work.

@pdr, is this problem related to
http://code.google.com/p/chromium/issues/detail?id=166164 ?

Thanks in Advance.

[pavane, 2013-09-13 00:52:18]

Please have a look at this patch

[pdr., 2013-09-13 05:15:44]

LGTM

[commit-bot: I haz the power, 2013-09-13 06:08:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pavan.e@samsung.com/22604002/4001

[commit-bot: I haz the power, 2013-09-13 07:16:00]

Change committed as 157739