| Summary: | Add support for injecting and rendering text recognition blocks | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Wenson Hsieh <wenson_hsieh> | ||||||||
| Component: | Platform | Assignee: | Wenson Hsieh <wenson_hsieh> | ||||||||
| Status: | RESOLVED FIXED | ||||||||||
| Severity: | Normal | CC: | akeerthi, cdumez, changseok, esprehn+autocc, ews-watchlist, gyuyoung.kim, hi, kangil.han, megan_gardner, thorton, webkit-bug-importer | ||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||
| Version: | WebKit Nightly Build | ||||||||||
| Hardware: | Unspecified | ||||||||||
| OS: | Unspecified | ||||||||||
| Attachments: |
|
||||||||||
|
Description
Wenson Hsieh
2021-11-12 08:01:01 PST
Created attachment 444073 [details]
Depends on #233010
Created attachment 444194 [details]
Minor adjustments / rebase on trunk
Comment on attachment 444194 [details] Minor adjustments / rebase on trunk View in context: https://bugs.webkit.org/attachment.cgi?id=444194&action=review r=me > Source/WebCore/dom/ImageOverlay.cpp:501 > + for (size_t index = 0; index < result.blocks.size(); ++index) { Should we add an assertion that `result.blocks.size() == elements.blocks.size()`? > Source/WebCore/dom/ImageOverlay.cpp:502 > + auto blockContainer = elements.blocks[index]; Nit: I would move this declaration after the `continue`, since it's not used otherwise. Comment on attachment 444194 [details] Minor adjustments / rebase on trunk View in context: https://bugs.webkit.org/attachment.cgi?id=444194&action=review Thanks for the review! >> Source/WebCore/dom/ImageOverlay.cpp:501 >> + for (size_t index = 0; index < result.blocks.size(); ++index) { > > Should we add an assertion that `result.blocks.size() == elements.blocks.size()`? Yes, added! >> Source/WebCore/dom/ImageOverlay.cpp:502 >> + auto blockContainer = elements.blocks[index]; > > Nit: I would move this declaration after the `continue`, since it's not used otherwise. Good catch — moved to after the continue. Created attachment 444386 [details]
Patch for landing
Committed r285862 (244288@main): <https://commits.webkit.org/244288@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 444386 [details]. |