Skip to content

Commit

Permalink
Avoid notifying PaintArtifactCompositor on VideoPainter
Browse files Browse the repository at this point in the history
Change-Id: I593c95441170d738565fc648b5e0029b6c88b554
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3776919
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Manuel Rego <rego@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1027126}
  • Loading branch information
mrego authored and Chromium LUCI CQ committed Jul 22, 2022
1 parent f21c367 commit 75844cd
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions third_party/blink/renderer/core/paint/video_painter.cc
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
#include "third_party/blink/renderer/core/paint/paint_info.h"
#include "third_party/blink/renderer/platform/geometry/layout_point.h"
#include "third_party/blink/renderer/platform/geometry/layout_rect.h"
#include "third_party/blink/renderer/platform/graphics/compositing/paint_artifact_compositor.h"
#include "third_party/blink/renderer/platform/graphics/paint/drawing_recorder.h"
#include "third_party/blink/renderer/platform/graphics/paint/foreign_layer_display_item.h"

Expand Down Expand Up @@ -65,13 +64,6 @@ void VideoPainter::PaintReplaced(const PaintInfo& paint_info,
snapped_replaced_rect);
}

// Since we may have changed the location of the replaced content, we need to
// notify PaintArtifactCompositor.
if (layout_video_.GetFrameView()) {
layout_video_.GetFrameView()->SetPaintArtifactCompositorNeedsUpdate(
PaintArtifactCompositorUpdateReason::kVideoPainterPaintReplaced);
}

// Video frames are only painted in software for printing or capturing node
// images via web APIs.
bool force_software_video_paint =
Expand Down

0 comments on commit 75844cd

Please sign in to comment.