Bug 205145

Summary: IndexedDB: make WebIDBServer a MessageReceiver to replace WebIDBConnection
Product: WebKit Reporter: Sihui Liu <sihui_liu>
Component: New BugsAssignee: Sihui Liu <sihui_liu>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, annulen, beidson, cdumez, commit-queue, ews-watchlist, gyuyoung.kim, ryuan.choi, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch for landing none

Sihui Liu
Reported 2019-12-11 16:48:59 PST
...
Attachments
Patch (60.62 KB, patch)
2019-12-11 16:53 PST, Sihui Liu 		no flags
DetailsFormatted DiffDiff
Patch for landing (61.03 KB, patch)
2019-12-12 15:11 PST, Sihui Liu 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Sihui Liu
Comment 1 2019-12-11 16:53:37 PST
Created attachment 385459 [details] Patch
Sihui Liu
Comment 2 2019-12-11 17:03:45 PST
Comment on attachment 385459 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=385459&action=review > Source/WebKit/ChangeLog:61 > + (WebKit::NetworkConnectionToWebProcess::establishIDBConnectionToServer): Deleted. NetworkConnectionToWebProcess::establishIDBConnectionToServer was used to ensure WebIDBConnectionToCilent member exists in NetworkConnectionToWebProcess. Now WebIDBConnectionToCilent is created in WebIDBServer::addConnection(when NetworkConnectionToWebProcess is created), so we don't need the message to do initialization.
Sihui Liu
Comment 3 2019-12-12 15:11:01 PST
Created attachment 385555 [details] Patch for landing
WebKit Commit Bot
Comment 4 2019-12-12 16:10:15 PST
Comment on attachment 385555 [details] Patch for landing Clearing flags on attachment: 385555 Committed r253452: <https://trac.webkit.org/changeset/253452>
WebKit Commit Bot
Comment 5 2019-12-12 16:10:17 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2019-12-12 16:11:23 PST
<rdar://problem/57895777>
Note You need to log in before you can comment on or make changes to this bug.