Bug 336046

Summary:	[testcase] Invalid css padding declaration should not be rendered
Product:	[Applications] konqueror	Reporter:	Andrea Iacovitti <aiacovitti>
Component:	khtml	Assignee:	Konqueror Bugs <konqueror-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	4.13.2
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:	http://commits.kde.org/kdelibs/bb2e6d6ebf8daf2d8fc838c5582204c7cfc3d1a6	Version Fixed/Implemented In:	4.13.3
Sentry Crash Report:
Attachments:	self explanatory testcase

Description Andrea Iacovitti 2014-06-10 14:15:55 UTC

Created attachment 87105 [details]
self explanatory testcase

The following padding shorthand declaration:
padding: 20px 20px boo
is invalid, thus it should be treated the same way as 'padding: 0px'.
khtml instead applies a padding-top/padding-right of 20px.

Comment 1 Andrea Iacovitti 2014-06-10 14:46:10 UTC

Git commit bb2e6d6ebf8daf2d8fc838c5582204c7cfc3d1a6 by Andrea Iacovitti.
Committed on 10/06/2014 at 14:34.
Pushed by aiacovitti into branch 'KDE/4.13'.

Remove subproperties previously added by parseValue() to our 'parsedProperties'
if the shorthand is invalid.
FIXED-IN: 4.13.3

M  +23   -4    khtml/css/cssparser.cpp
M  +1    -0    khtml/css/cssparser.h

http://commits.kde.org/kdelibs/bb2e6d6ebf8daf2d8fc838c5582204c7cfc3d1a6