fix: leaked gfx::Canvas in AutofillPopupView::OnPaint() (#46384)

* perf: avoid redundant call to popup_bounds_in_view()

* refactor: use a std::optional<> for paint_canvas local

* fix: fix leaked gfx::Canvas in AutofillPopupView::OnPaint()

* refactor: remove redundant get() call when testing smart pointer for nonempty

* refactor: remove unnecessary draw_canvas variable

* refactor: rename bitmap to offscreen_bitmap for symmetry

* refactor: avoid another redundant call to popup_bounds_in_view()
This commit is contained in:
Charles Kerr 2025-04-01 11:22:26 -05:00 committed by GitHub
parent 5a6f1ede6a
commit ac1ffb1bff
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -5,6 +5,7 @@
#include "shell/browser/ui/views/autofill_popup_view.h"
#include <memory>
#include <optional>
#include <utility>
#include "base/functional/bind.h"
@ -237,30 +238,33 @@ void AutofillPopupView::DoUpdateBoundsAndRedrawPopup() {
void AutofillPopupView::OnPaint(gfx::Canvas* canvas) {
if (!popup_ || static_cast<size_t>(popup_->line_count()) != children().size())
return;
gfx::Canvas* draw_canvas = canvas;
SkBitmap bitmap;
std::unique_ptr<cc::SkiaPaintCanvas> paint_canvas;
if (view_proxy_.get()) {
bitmap.allocN32Pixels(popup_->popup_bounds_in_view().width(),
popup_->popup_bounds_in_view().height(), true);
paint_canvas = std::make_unique<cc::SkiaPaintCanvas>(bitmap);
draw_canvas = new gfx::Canvas(paint_canvas.get(), 1.0);
gfx::Rect offscreen_bounds;
SkBitmap offscreen_bitmap;
std::optional<cc::SkiaPaintCanvas> offscreen_paint_canvas;
std::optional<gfx::Canvas> offscreen_draw_canvas;
if (view_proxy_) {
offscreen_bounds = popup_->popup_bounds_in_view();
offscreen_bitmap.allocN32Pixels(offscreen_bounds.width(),
offscreen_bounds.height(), true);
offscreen_paint_canvas.emplace(offscreen_bitmap);
offscreen_draw_canvas.emplace(&offscreen_paint_canvas.value(), 1.0);
canvas = &offscreen_draw_canvas.value();
}
draw_canvas->DrawColor(
canvas->DrawColor(
GetColorProvider()->GetColor(ui::kColorResultsTableNormalBackground));
OnPaintBorder(draw_canvas);
OnPaintBorder(canvas);
for (int i = 0; i < popup_->line_count(); ++i) {
gfx::Rect line_rect = popup_->GetRowBounds(i);
DrawAutofillEntry(draw_canvas, i, line_rect);
DrawAutofillEntry(canvas, i, line_rect);
}
if (view_proxy_.get()) {
view_proxy_->SetBounds(popup_->popup_bounds_in_view());
view_proxy_->SetBitmap(bitmap);
if (view_proxy_) {
view_proxy_->SetBounds(offscreen_bounds);
view_proxy_->SetBitmap(offscreen_bitmap);
}
}