Bug 266552
| Summary: | Fix hang in SQLTransactionCoordinator::shutdown() | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ap, cdumez, mark.lam, sihui_liu, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
Hi Team,
While going through potential merges from Blink, I came across another potential:
Blink Commit: https://src.chromium.org/viewvc/blink?view=revision&revision=181269
WebKit Source: https://searchfox.org/wubkat/rev/9222f67347ee90fcc400687e756ef8c8979748f0/Source/WebCore/Modules/webdatabase/SQLTransactionCoordinator.cpp#132
Here, it should be:
RefPtr<SQLTransaction> transaction = info.pendingTransactions.takeFirst();
Current, Chromium / Blink source also has above patch applied:
https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/modules/webdatabase/sql_transaction_coordinator.cc;l=143;drc=fb656ef1e239adc1ab9960e8d7916a7f6df4409a
Just wanted to raise, so we can import this and fix any issue.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
EWS
Committed 272316@main (4a40fea04d2f): <https://commits.webkit.org/272316@main>
Reviewed commits have been landed. Closing PR #22032 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/119903159>