tree 2cd54ad7743aa3a50520e2858ca83f1488bf9312
parent 19c2bebe7d521d945f0430d18df076cab1a3962e
author Justin Novosad <junov@chromium.org> 1623867151 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1623867151 +0000

[Canvas] Simplify DidDraw methods (Reland)

This is a code cleanup related to DidDraw methods throughout
canvas-related code in blink. This CL removes unnecessary overloads,
unnecessary virtuals, unnecessary argument passing. In WebGL
rendering contexts, we remove an unnecessary recursion cycle. In
2D contexts, we remove an unnecessary and confusing virtual method
multiple inherritance name conflict (method renamed to DidDraw2D
in BaseRenderingContext2D)

Difference from original CL that was reverted: removed changes
to dirty_rect calulations in html_canvas_element.cc

Original CL:
https://chromium-review.googlesource.com/c/chromium/src/+/2947051/

Bug: 1217714
Change-Id: I98a2b87418389adfe368a2a7cb2c1175e3cc221e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2965277
Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Reviewed-by: Xida Chen <xidachen@chromium.org>
Reviewed-by: Fernando Serboncini <fserb@chromium.org>
Commit-Queue: Justin Novosad <junov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#893061}