[ImageDecoder] Remove decodeMetadata() in favor of tracks.ready. #28612
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since metadata is always queued automatically, decodeMetadata()
doesn't really do what it says. There's a better web idiom for
this a 'ready' property that's a one-time resolved promise.
Instead of calling decodeMetadata(), clients should just use
decode.tracks.ready.then(...)
Since the last round of breaking changes didn't cause any
issues without a deprecation period, drop decodeMetadata()
without a deprecation period as well.
Fixed: 1182435
Change-Id: I95317d6daaba9a98a7f33d594b50921800a6fbe3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2842034
Auto-Submit: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Chrome Cunningham <chcunningham@chromium.org>
Reviewed-by: Chrome Cunningham <chcunningham@chromium.org>
Cr-Commit-Position: refs/heads/master@{#874987}