Title: [193560] branches/safari-601.1.46-branch/Source/WebCore
- Revision
- 193560
- Author
- [email protected]
- Date
- 2015-12-05 22:24:18 -0800 (Sat, 05 Dec 2015)
Log Message
Merged r191825. rdar://problem/23221179
Modified Paths
Diff
Modified: branches/safari-601.1.46-branch/Source/WebCore/ChangeLog (193559 => 193560)
--- branches/safari-601.1.46-branch/Source/WebCore/ChangeLog 2015-12-06 06:23:15 UTC (rev 193559)
+++ branches/safari-601.1.46-branch/Source/WebCore/ChangeLog 2015-12-06 06:24:18 UTC (rev 193560)
@@ -1,5 +1,20 @@
2015-12-05 Babak Shafiei <[email protected]>
+ Merge r191825.
+
+ 2015-10-30 Joseph Pecoraro <[email protected]>
+
+ CSSParserVariable leaks seen on leaks bots
+ https://bugs.webkit.org/show_bug.cgi?id=150724
+
+ Reviewed by Darin Adler.
+
+ * css/CSSParserValues.cpp:
+ (WebCore::destroy):
+ Cleanup variable CSSParserValues.
+
+2015-12-05 Babak Shafiei <[email protected]>
+
Merge r191128.
2015-10-14 David Hyatt <[email protected]>
Modified: branches/safari-601.1.46-branch/Source/WebCore/css/CSSGrammar.y.in (193559 => 193560)
--- branches/safari-601.1.46-branch/Source/WebCore/css/CSSGrammar.y.in 2015-12-06 06:23:15 UTC (rev 193559)
+++ branches/safari-601.1.46-branch/Source/WebCore/css/CSSGrammar.y.in 2015-12-06 06:24:18 UTC (rev 193560)
@@ -106,7 +106,7 @@
#if ENABLE_CSS_GRID_LAYOUT
%expect 39
#else
-%expect 37
+%expect 38
#endif
%nonassoc LOWEST_PREC
Modified: branches/safari-601.1.46-branch/Source/WebCore/css/CSSParser.cpp (193559 => 193560)
--- branches/safari-601.1.46-branch/Source/WebCore/css/CSSParser.cpp 2015-12-06 06:23:15 UTC (rev 193559)
+++ branches/safari-601.1.46-branch/Source/WebCore/css/CSSParser.cpp 2015-12-06 06:24:18 UTC (rev 193560)
@@ -4154,12 +4154,11 @@
if (m_customPropertyName.isEmpty() || !m_valueList)
return false;
- auto& cssValuePool = CSSValuePool::singleton();
RefPtr<CSSValue> value;
if (id == CSSValueInherit)
- value = cssValuePool.createInheritedValue();
+ value = cssValuePool().createInheritedValue();
else if (id == CSSValueInitial)
- value = cssValuePool.createExplicitInitialValue();
+ value = cssValuePool().createExplicitInitialValue();
else {
RefPtr<CSSValueList> valueList = CSSValueList::createFromParserValueList(*m_valueList);
if (m_valueList->containsVariables())
Modified: branches/safari-601.1.46-branch/Source/WebCore/css/CSSParserValues.cpp (193559 => 193560)
--- branches/safari-601.1.46-branch/Source/WebCore/css/CSSParserValues.cpp 2015-12-06 06:23:15 UTC (rev 193559)
+++ branches/safari-601.1.46-branch/Source/WebCore/css/CSSParserValues.cpp 2015-12-06 06:24:18 UTC (rev 193560)
@@ -40,6 +40,8 @@
delete value.function;
else if (value.unit == CSSParserValue::ValueList)
delete value.valueList;
+ else if (value.unit == CSSParserValue::Variable)
+ delete value.variable;
}
CSSParserValueList::~CSSParserValueList()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes