Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(117)

Side by Side Diff: Source/core/svg/SVGAnimationElement.cpp

Issue 302643004: [SVG2] Allow leading and trailing whitespace in svg attributes (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@relax_todouble_wtf
Patch Set: review fixes Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2004, 2005 Nikolas Zimmermann <zimmermann@kde.org> 2 * Copyright (C) 2004, 2005 Nikolas Zimmermann <zimmermann@kde.org>
3 * Copyright (C) 2004, 2005, 2006, 2007 Rob Buis <buis@kde.org> 3 * Copyright (C) 2004, 2005, 2006, 2007 Rob Buis <buis@kde.org>
4 * Copyright (C) 2007 Eric Seidel <eric@webkit.org> 4 * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
5 * Copyright (C) 2008 Apple Inc. All rights reserved. 5 * Copyright (C) 2008 Apple Inc. All rights reserved.
6 * Copyright (C) 2009 Cameron McCormack <cam@mcc.id.au> 6 * Copyright (C) 2009 Cameron McCormack <cam@mcc.id.au>
7 * Copyright (C) Research In Motion Limited 2010. All rights reserved. 7 * Copyright (C) Research In Motion Limited 2010. All rights reserved.
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 return; 103 return;
104 } 104 }
105 105
106 float posC = 0; 106 float posC = 0;
107 if (!parseNumber(ptr, end, posC)) { 107 if (!parseNumber(ptr, end, posC)) {
108 result.clear(); 108 result.clear();
109 return; 109 return;
110 } 110 }
111 111
112 float posD = 0; 112 float posD = 0;
113 if (!parseNumber(ptr, end, posD, false)) { 113 if (!parseNumber(ptr, end, posD, STRICT)) {
114 result.clear(); 114 result.clear();
115 return; 115 return;
116 } 116 }
117 117
118 skipOptionalSVGSpaces(ptr, end); 118 skipOptionalSVGSpaces(ptr, end);
119 119
120 if (ptr < end && *ptr == ';') { 120 if (ptr < end && *ptr == ';') {
121 delimParsed = true; 121 delimParsed = true;
122 ptr++; 122 ptr++;
123 } 123 }
(...skipping 585 matching lines...) Expand 10 before | Expand all | Expand 10 after
709 SVGSMILElement::setAttributeName(attributeName); 709 SVGSMILElement::setAttributeName(attributeName);
710 checkInvalidCSSAttributeType(targetElement()); 710 checkInvalidCSSAttributeType(targetElement());
711 } 711 }
712 712
713 void SVGAnimationElement::checkInvalidCSSAttributeType(SVGElement* target) 713 void SVGAnimationElement::checkInvalidCSSAttributeType(SVGElement* target)
714 { 714 {
715 m_hasInvalidCSSAttributeType = target && hasValidAttributeName() && attribut eType() == AttributeTypeCSS && !isTargetAttributeCSSProperty(target, attributeNa me()); 715 m_hasInvalidCSSAttributeType = target && hasValidAttributeName() && attribut eType() == AttributeTypeCSS && !isTargetAttributeCSSProperty(target, attributeNa me());
716 } 716 }
717 717
718 } 718 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698