[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdelibs/khtml/css
From:       Maks Orlovich <maksim () kde ! org>
Date:       2008-09-10 0:02:44
Message-ID: 1221004964.464514.7467.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 859306 by orlovich:

Merged revision 859305:
Fix a regression in background: shorthand parsing:
parseBackgroundPositionXY was impropertly setting the out position 
when it didn't match, which incorrect classified the position of a missing 2nd 
coordinate in a shorthard as non-keyword, hence dropping the rule if the 
first one was vertical
BUG:170755

 M  +4 -2      cssparser.cpp  


--- trunk/KDE/kdelibs/khtml/css/cssparser.cpp #859305:859306
@@ -1501,10 +1501,12 @@
         }
         return new CSSPrimitiveValueImpl(percent, CSSPrimitiveValue::CSS_PERCENTAGE);
     }
-    kindOut = BgPos_NonKW;
-    if (validUnit(valueList->current(), FPercent|FLength, strict))
+
+    if (validUnit(valueList->current(), FPercent|FLength, strict)) {
+        kindOut = BgPos_NonKW;
         return new CSSPrimitiveValueImpl(valueList->current()->fValue,
                                          (CSSPrimitiveValue::UnitTypes)valueList->current()->unit);
+    }
 
     return 0;
 }
[prev in list] [next in list] [prev in thread] [next in thread]