Commit c2a9e4b8 authored by Xiaocheng Hu's avatar Xiaocheng Hu Committed by Commit Bot

Clean up an Oilpan-related TODO(825895)

As discussed offline, this usage is Oilpan-safe. As I had a few other
attempts without making it simpler (e.g., without using Member<>), I'll
simply remove this TODO.

Bug: 825895
Change-Id: Id677e59625d09a9888e98fac853da734d9d83669
NoTry: True
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1832528Reviewed-by: default avatarEmil A Eklund <eae@chromium.org>
Reviewed-by: default avatarMichael Lippautz <mlippautz@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#702065}
parent 1d421a0a
...@@ -324,7 +324,6 @@ class CSSMathExpressionVariadicOperation final : public CSSMathExpressionNode { ...@@ -324,7 +324,6 @@ class CSSMathExpressionVariadicOperation final : public CSSMathExpressionNode {
private: private:
// Helper for iterating from the 2nd to the last operands // Helper for iterating from the 2nd to the last operands
// TODO(crbug.com/825895): Is this Oilpan-safe?
base::span<const Member<const CSSMathExpressionNode>> SecondToLastOperands() base::span<const Member<const CSSMathExpressionNode>> SecondToLastOperands()
const { const {
return base::make_span(std::next(operands_.begin()), operands_.end()); return base::make_span(std::next(operands_.begin()), operands_.end());
......
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