Bug 274035
| Summary: | Adopt SafeTextMarkerData in more places | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Chris Dumez <cdumez> |
| Component: | Accessibility | Assignee: | Diego Pino <dpino> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | andresg_22, dpino, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Chris Dumez
Adopt SafeTextMarkerData in more places.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/127935353>
Chris Dumez
Pull request: https://github.com/WebKit/WebKit/pull/28435
EWS
Committed 278666@main (c0cc24bd0684): <https://commits.webkit.org/278666@main>
Reviewed commits have been landed. Closing PR #28435 and removing active labels.
Diego Pino
Build bot 'GTK-Linux-64-bit-Release-Debian-11-Build' if failing after this change with the following error:
https://build.webkit.org/#/builders/992/builds/2218
```
./../../Source/WebCore/accessibility/AXTextMarker.cpp:157:28: error: ‘auto’ parameter not permitted in this context
157 | CharacterOffset result(RefPtr { m_data.node.get() }.get(), m_data.characterStart, m_data.characterOffset);
| ^~~~~~
```
Diego Pino
Re-opening for pull request https://github.com/webkit/webkit/pull/28459
EWS
Committed 278677@main (c7adbf8ae9c0): <https://commits.webkit.org/278677@main>
Reviewed commits have been landed. Closing PR #28459 and removing active labels.