Add createImageFromTextureSource method to ui_web

[jezell]

Adds createImageFromTextureSource for flutter web, exposed in dart:web_ui.

final ui.Image uiImage = renderer.createImageFromTextureSource(bitmap,
            width: 150, height: 150, transferOwnership: false);

In canvaskit renderer, this will use MakeLazyImageFromTextureSource. In SkWasmRenderer, it will call imageCreateFromTextureSource, which was already implemented in SkWasm for createImageFromImageBitmap to use, but was not exposed in a way that it could be taken advantage of.

By default, createImageFromTextureSource will create a copy of the object it is passed, but transferOwnership: true may be specified to allow transferable objects to be transferred to the renderer, avoiding the copy.

Fixes: flutter/flutter#150479
Fixes: flutter/flutter#144815

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[kevmoo]

@jezell – look at a rebase and failing tests?

[kevmoo]

@jezell Are the impeller changes related?

[eyebrowsoffire]

This looks like it has pulled in a bunch of unrelated changes. Can you clean this PR up so that only contains the changes related to the proposed createImageFromTextureSource API?

[flutter-dashboard]

Golden file changes have been found for this pull request. Click here to view and triage (e.g. because this is an intentional change).

If you are still iterating on this change and are not ready to resolve the images on the Flutter Gold dashboard, consider marking this PR as a draft pull request above. You will still be able to view image results on the dashboard, commenting will be silenced, and the check will not try to resolve itself until marked ready for review.

Changes reported for pull request #53483 at sha 7d71661

[eyebrowsoffire]

Looks good to me, modulo a few random formatting things, and one requested change to the API surface. Thanks for the fix! I think this will be a good addition.

[flutter-dashboard]

Golden file changes are available for triage from new commit, Click here to view.

Changes reported for pull request #53483 at sha c1ad286

[flutter-dashboard]

Golden file changes are available for triage from new commit, Click here to view.

Changes reported for pull request #53483 at sha 68fe742

[eyebrowsoffire]

I approved the new goldens.

[kevmoo]

Okay to auto-submit @eyebrowsoffire ?

[auto-submit]

auto label is removed for flutter/engine/53483, due to This PR has not met approval requirements for merging. The PR author is not a member of flutter-hackers and needs 1 more review(s) in order to merge this PR.

• Merge guidelines: A PR needs at least one approved review if the author is already part of flutter-hackers or two member reviews if the author is not a flutter-hacker before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

[jezell]

Thanks @kevmoo @eyebrowsoffire