Fix layering violation in ScrollbarPainter |
|
Issue descriptionhttps://cs.chromium.org/chromium/src/third_party/blink/renderer/core/paint/scrollbar_painter.cc?rcl=4a458a69ceb15f48bf104437f900992875f5c5b0&l=31: // Make sure our dimensions match the rect. // FIXME: Setting these is a bad layering violation! const_cast<LayoutScrollbarPart&>(layout_scrollbar_part) .SetLocation(rect.Location() - ToSize(paint_offset)); const_cast<LayoutScrollbarPart&>(layout_scrollbar_part) .SetWidth(rect.Width()); const_cast<LayoutScrollbarPart&>(layout_scrollbar_part) .SetHeight(rect.Height()); This is bad, and also prevents us from generate correct paint offset for the scrollbar parts during PrePaint. |
|
►
Sign in to add a comment |
|