Commit 96e04a93 authored by Xiaocheng Hu's avatar Xiaocheng Hu Committed by Chromium LUCI CQ

Clamp overflow OL counter values

Bug: 1161232
Change-Id: I022b51133370062c9419198f44e2a5723256825e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2613364Reviewed-by: default avatarIan Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#840769}
parent c35c0718
...@@ -24,6 +24,7 @@ ...@@ -24,6 +24,7 @@
#include <memory> #include <memory>
#include "base/memory/ptr_util.h" #include "base/memory/ptr_util.h"
#include "base/numerics/clamped_math.h"
#include "third_party/blink/renderer/core/css/counter_style.h" #include "third_party/blink/renderer/core/css/counter_style.h"
#include "third_party/blink/renderer/core/dom/element.h" #include "third_party/blink/renderer/core/dom/element.h"
#include "third_party/blink/renderer/core/dom/element_traversal.h" #include "third_party/blink/renderer/core/dom/element_traversal.h"
...@@ -204,8 +205,8 @@ bool PlanCounter(LayoutObject& object, ...@@ -204,8 +205,8 @@ bool PlanCounter(LayoutObject& object,
return true; return true;
} }
if (auto* olist = DynamicTo<HTMLOListElement>(*e)) { if (auto* olist = DynamicTo<HTMLOListElement>(*e)) {
value = value = base::ClampAdd(olist->StartConsideringItemCount(),
olist->StartConsideringItemCount() + (olist->IsReversed() ? 1 : -1); olist->IsReversed() ? 1 : -1);
type_mask = CounterNode::kResetType; type_mask = CounterNode::kResetType;
return true; return true;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment