commit 8261abbf56712d2e2812cd4ceaa601f20e16098a
author Yuki Shiino <yukishiino@chromium.org> Wed Aug 14 02:45:13 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Aug 14 02:45:13 2024
tree 94cab8f13172b0bdfd2c3299d7ab7798760f15b4
parent a7f158afee1204289d6e29bc91e961da06c353bb

Revert "[lensoverlay] Pass bitmap directly to WebUI instead of encoding to JPEG"

This reverts commit 71f921cac550fcc4c45855c3830dd1f00bb6ad79.

Reason for revert: Suspicious about causing test failures of
LensOverlayTest.TextSelection
LensOverlayControllerSearchBubbleDisabled.PreselectionToastDisappearsOnSelection
https://ci.chromium.org/ui/p/chromium/builders/ci/Mac11%20Tests/30294/overview

Original change's description:
> [lensoverlay] Pass bitmap directly to WebUI instead of encoding to JPEG
>
> Passing the bitmap directly to the WebUI, helps in a couple ways.
>   1) It moves the encoding to JPEG into the renderer process, therefore
>      removing it from the critical path on the main thread.
>   2) Passes the pixels as shared memory to the WebUI.
>   3) Removes the need for data URIs.
>
> With this approach, we saw slight wins on M1 macbooks, and significant
> wins (25% improvement) in the time it takes to go from capturing the
> screenshot to rendering it to the user. This approach also sets up
> an additional win by drawing the bitmap and rendering directly in the
> JS canvas, which should be a big win, since the bottleneck in this flow
> is the call to toBlob().
>
> This is step 1 of many to come CLs to improve our invocation time. See
> bug for more details.
>
> NO_IFTTT= New line added by git cl format
>
> Bug: b:352622136
> Change-Id: I25573e2600a9ae7a0b1b05133f03d65e6293f7b2
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5770332
> Reviewed-by: Erik Chen <erikchen@chromium.org>
> Commit-Queue: Duncan Mercer <mercerd@google.com>
> Cr-Commit-Position: refs/heads/main@{#1341186}

Bug: b:352622136
Change-Id: I828b4e351db55f01e92f53ee2affd9ccac60cedb
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5782680
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Yuki Shiino <yukishiino@chromium.org>
Owners-Override: Yuki Shiino <yukishiino@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1341425}