Fixup reference for svg/animations/viewspec-checkaspectparams.html

Fixup reference for svg/animations/viewspec-checkaspectparams.html

Use the expected 'viewBox' and 'preserveAspectRatio' instead, and
modify the <rect> to match what's in the resources/ svg-file.
Drop a redundant semicolon from the view-spec (this is not a parsing
test).

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

[fs, 2015-05-08 08:31:14]

fs@opera.com changed reviewers:
+ davve@opera.com, wangxianzhu@chromium.org

[davve, 2015-05-08 11:48:57]

Non-owner LGTM

Test (still) present in TestExpectations. Intended?

[fs, 2015-05-08 11:55:33]

On 2015/05/08 11:48:57, David Vest wrote:
> Non-owner LGTM

In LayoutTests/ "everybody" is an owener ;-)
 
> Test (still) present in TestExpectations. Intended?

I forgot to include the context:
https://codereview.chromium.org/1098273002/#msg22

(Still no idea how it managed to pass CQ in the first place...)

[davve, 2015-05-08 12:06:24]

On 2015/05/08 11:55:33, fs wrote:
> On 2015/05/08 11:48:57, David Vest wrote:
> > Non-owner LGTM
> 
> In LayoutTests/ "everybody" is an owener ;-)

Right. Owner LGTM then :)

> > Test (still) present in TestExpectations. Intended?
> 
> I forgot to include the context:
> https://codereview.chromium.org/1098273002/#msg22
> 
> (Still no idea how it managed to pass CQ in the first place...)

Thanks.

[fs, 2015-05-08 12:15:16]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2015-05-08 12:15:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1131233002/1

[commit-bot: I haz the power, 2015-05-08 12:19:20]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=195110

[Xianzhu, 2015-05-08 15:56:45]

On 2015/05/08 11:55:33, fs wrote:
> On 2015/05/08 11:48:57, David Vest wrote:
> > Non-owner LGTM
> 
> In LayoutTests/ "everybody" is an owener ;-)
>  
> > Test (still) present in TestExpectations. Intended?
> 
> I forgot to include the context:
> https://codereview.chromium.org/1098273002/#msg22
> 
> (Still no idea how it managed to pass CQ in the first place...)

It seems that some layout update and/or painting in the test is scheduled in
next cycle. The issue was exposed when I changed the final pixel dump to
asynchronous. (https://codereview.chromium.org/1127013005/).