Title: [172985] trunk/Source/WebCore
Revision
172985
Author
benja...@webkit.org
Date
2014-08-26 17:01:12 -0700 (Tue, 26 Aug 2014)

Log Message

Minor style tweak on StyleResolveTree
https://bugs.webkit.org/show_bug.cgi?id=136233

Patch by Benjamin Poulain <bpoul...@apple.com> on 2014-08-26
Reviewed by Sam Weinig.

* style/StyleResolveTree.cpp:
(WebCore::Style::resolveTree):
Clang likely optimized it away, but I find that explicit style cleaner.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (172984 => 172985)


--- trunk/Source/WebCore/ChangeLog	2014-08-26 23:59:29 UTC (rev 172984)
+++ trunk/Source/WebCore/ChangeLog	2014-08-27 00:01:12 UTC (rev 172985)
@@ -1,3 +1,14 @@
+2014-08-26  Benjamin Poulain  <bpoul...@apple.com>
+
+        Minor style tweak on StyleResolveTree
+        https://bugs.webkit.org/show_bug.cgi?id=136233
+
+        Reviewed by Sam Weinig.
+
+        * style/StyleResolveTree.cpp:
+        (WebCore::Style::resolveTree):
+        Clang likely optimized it away, but I find that explicit style cleaner.
+
 2014-08-26  Brian J. Burg  <b...@cs.washington.edu>
 
         Web Inspector: put feature flags for Inspector domains in the protocol specification

Modified: trunk/Source/WebCore/style/StyleResolveTree.cpp (172984 => 172985)


--- trunk/Source/WebCore/style/StyleResolveTree.cpp	2014-08-26 23:59:29 UTC (rev 172984)
+++ trunk/Source/WebCore/style/StyleResolveTree.cpp	2014-08-27 00:01:12 UTC (rev 172985)
@@ -931,8 +931,8 @@
         bool forceCheckOfNextElementSibling = false;
         bool forceCheckOfAnyElementSibling = false;
         for (Node* child = current.firstChild(); child; child = child->nextSibling()) {
-            if (child->renderer())
-                childRenderTreePosition.invalidateNextSibling(*child->renderer());
+            if (RenderObject* childRenderer = child->renderer())
+                childRenderTreePosition.invalidateNextSibling(*childRenderer);
             if (child->isTextNode() && child->needsStyleRecalc()) {
                 resolveTextNode(*toText(child), childRenderTreePosition);
                 continue;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to