Title: [175438] trunk/Source/WebCore
- Revision
- 175438
- Author
- [email protected]
- Date
- 2014-10-31 16:38:12 -0700 (Fri, 31 Oct 2014)
Log Message
Brace cleanup in StyleResolver::adjustRenderStyle()
https://bugs.webkit.org/show_bug.cgi?id=138267
Reviewed by Chris Dumez.
No braces for single-line clauses.
* css/StyleResolver.cpp:
(WebCore::StyleResolver::adjustRenderStyle):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (175437 => 175438)
--- trunk/Source/WebCore/ChangeLog 2014-10-31 23:37:35 UTC (rev 175437)
+++ trunk/Source/WebCore/ChangeLog 2014-10-31 23:38:12 UTC (rev 175438)
@@ -1,3 +1,15 @@
+2014-10-31 Simon Fraser <[email protected]>
+
+ Brace cleanup in StyleResolver::adjustRenderStyle()
+ https://bugs.webkit.org/show_bug.cgi?id=138267
+
+ Reviewed by Chris Dumez.
+
+ No braces for single-line clauses.
+
+ * css/StyleResolver.cpp:
+ (WebCore::StyleResolver::adjustRenderStyle):
+
2014-10-31 Chris Dumez <[email protected]>
Fix several warnings reported by clang static analyzer in WebCore
Modified: trunk/Source/WebCore/css/StyleResolver.cpp (175437 => 175438)
--- trunk/Source/WebCore/css/StyleResolver.cpp 2014-10-31 23:37:35 UTC (rev 175437)
+++ trunk/Source/WebCore/css/StyleResolver.cpp 2014-10-31 23:38:12 UTC (rev 175438)
@@ -1341,9 +1341,8 @@
}
// Let the theme also have a crack at adjusting the style.
- if (style.hasAppearance()) {
+ if (style.hasAppearance())
RenderTheme::defaultTheme()->adjustStyle(*this, style, e, m_state.hasUAAppearance(), m_state.borderData(), m_state.backgroundData(), m_state.backgroundColor());
- }
// If we have first-letter pseudo style, do not share this style.
if (style.hasPseudoStyle(FIRST_LETTER))
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
