commit a0168cb154ece8428daf50b0557a1f85bceddfb8
author Asami Doi <asamidoi@chromium.org> Tue Nov 28 18:24:50 2023
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Nov 28 18:24:50 2023
tree 759e8129cdf620935f5d5570df9f2e8fc8c899ad
parent 4e066ae4e907f4e2d675f0983b807a7d5884bc53

[ios] Remove outdated TODO comments

This CL removes outdated TODO comments because the issue associated
to the comment is closed and we need the workaround.

Change-Id: Ife00f0ca0826eebfad380da2f2758471db158cd4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5064153
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Asami Doi <asamidoi@chromium.org>
Auto-Submit: Asami Doi <asamidoi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1230018}

ios/chrome/browser/snapshots/model/snapshot_generator.mm

diff --git a/ios/chrome/browser/snapshots/model/snapshot_generator.mm b/ios/chrome/browser/snapshots/model/snapshot_generator.mm
index abce80c50..b332405 100644
--- a/ios/chrome/browser/snapshots/model/snapshot_generator.mm
+++ b/ios/chrome/browser/snapshots/model/snapshot_generator.mm
@@ -4,9 +4,8 @@
 
 #import "ios/chrome/browser/snapshots/model/snapshot_generator.h"
 
-// TODO(crbug.com/636188): required to implement ViewHierarchyContainsWebView
-// for -drawViewHierarchyInRect:afterScreenUpdates:, remove once the workaround
-// is no longer needed.
+// It is required to implement ViewHierarchyContainsWebView to check if
+// the view is `WKWebView`.
 #import <WebKit/WebKit.h>
 
 #import "base/functional/bind.h"
@@ -189,10 +188,6 @@
           // called on a view that is part of view controller containment and
           // not in the view hierarchy, an
           // UIViewControllerHierarchyInconsistency exception will be thrown.
-          // TODO(crbug.com/636188):
-          // `-drawViewHierarchyInRect:afterScreenUpdates:` is buggy causing GPU
-          // glitches, screen redraws during animations, broken pinch to dismiss
-          // on tablet, etc.
           snapshotSuccess = [baseView drawViewHierarchyInRect:baseView.bounds
                                            afterScreenUpdates:YES];
         } else {