summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAllan Sandfeld Jensen <allan.jensen@qt.io>2020-08-03 17:38:58 +0200
committerAllan Sandfeld Jensen <allan.jensen@qt.io>2020-08-03 15:47:28 +0000
commit05942008bf7a5ca403b2e33901905dba6e78acc9 (patch)
tree61a20e2da0da0e9e9e72713e1557b71ae5130df9
parent91ec8de034384375e9e63e6694a09948ce4941f2 (diff)
downloadqtwebengine-chromium-05942008bf7a5ca403b2e33901905dba6e78acc9.tar.gz
[Backport] Handle invalid type mixing in CSS clamp()
Bug: 1050608 Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#740710} Fixes: QTBUG-85863 Change-Id: If05d806be6ff35c209c983ef8d79790ecd1addb7 Reviewed-by: Michael BrĂ¼ning <michael.bruning@qt.io>
-rw-r--r--chromium/third_party/blink/renderer/core/css/css_math_expression_node.cc6
1 files changed, 6 insertions, 0 deletions
diff --git a/chromium/third_party/blink/renderer/core/css/css_math_expression_node.cc b/chromium/third_party/blink/renderer/core/css/css_math_expression_node.cc
index 4e6fa5e20e4..f5fe60cb8a6 100644
--- a/chromium/third_party/blink/renderer/core/css/css_math_expression_node.cc
+++ b/chromium/third_party/blink/renderer/core/css/css_math_expression_node.cc
@@ -1083,8 +1083,14 @@ class CSSMathExpressionNodeParser {
auto* nested = CSSMathExpressionVariadicOperation::Create(
{val_operand, max_operand}, CSSMathOperator::kMin);
+ if (!nested)
+ return nullptr;
+
auto* result = CSSMathExpressionVariadicOperation::Create(
{min_operand, nested}, CSSMathOperator::kMax);
+ if (!result)
+ return nullptr;
+
result->SetIsClamp();
return result;
}