Title: [113854] trunk/Source/WebCore
Revision
113854
Author
macpher...@chromium.org
Date
2012-04-11 06:29:01 -0700 (Wed, 11 Apr 2012)

Log Message

Remove unnecessary assignments from CSSParser::parseLinearGradient().
https://bugs.webkit.org/show_bug.cgi?id=83661

Reviewed by Kentaro Hara.

No new tests / code cleanup only.

* css/CSSParser.cpp:
(WebCore::CSSParser::parseLinearGradient):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (113853 => 113854)


--- trunk/Source/WebCore/ChangeLog	2012-04-11 13:23:36 UTC (rev 113853)
+++ trunk/Source/WebCore/ChangeLog	2012-04-11 13:29:01 UTC (rev 113854)
@@ -1,3 +1,15 @@
+2012-04-11  Luke Macpherson  <macpher...@chromium.org>
+
+        Remove unnecessary assignments from CSSParser::parseLinearGradient().
+        https://bugs.webkit.org/show_bug.cgi?id=83661
+
+        Reviewed by Kentaro Hara.
+
+        No new tests / code cleanup only.
+
+        * css/CSSParser.cpp:
+        (WebCore::CSSParser::parseLinearGradient):
+
 2012-04-11  Stephen Chenney  <schen...@chromium.org>
 
         Shadow tree TreeScope data is not removed by ContainerNode::removeAllChildren

Modified: trunk/Source/WebCore/css/CSSParser.cpp (113853 => 113854)


--- trunk/Source/WebCore/css/CSSParser.cpp	2012-04-11 13:23:36 UTC (rev 113853)
+++ trunk/Source/WebCore/css/CSSParser.cpp	2012-04-11 13:29:01 UTC (rev 113854)
@@ -6389,7 +6389,7 @@
     if (validUnit(a, FAngle, CSSStrictMode)) {
         result->setAngle(createPrimitiveNumericValue(a));
 
-        a = args->next();
+        args->next();
         expectComma = true;
     } else {
         // Look one or two optional keywords that indicate a side or corner.
@@ -6403,8 +6403,7 @@
             else
                 startY = location;
 
-            a = args->next();
-            if (a) {
+            if ((a = args->next())) {
                 if ((location = valueFromSideKeyword(a, isHorizontal))) {
                     if (isHorizontal) {
                         if (startX)
@@ -6416,7 +6415,7 @@
                         startY = location;
                     }
 
-                    a = args->next();
+                    args->next();
                 }
             }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to