WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
259715
Merge ResourceLoadStatisticsStore and ResourceLoadStatisticsDatabaseStore
https://bugs.webkit.org/show_bug.cgi?id=259715
Summary
Merge ResourceLoadStatisticsStore and ResourceLoadStatisticsDatabaseStore
Brent Fulgham
Reported
2023-08-01 16:50:28 PDT
Now that we only have a SQLite-based ITP database, we should remove the needless layering created by a separate ResourceLoadStatisticsStore and ResourceLoadStatisticsDatabaseStore child class. This will allow us to reduce complexity by removing a needless child class, but will also remove numerous virtual method calls, type checks, and downcast operations needed to dispatch methods to the child class.
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2023-08-01 16:50:48 PDT
<
rdar://problem/113236158
>
Brent Fulgham
Comment 2
2023-08-01 17:10:40 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/16298
EWS
Comment 3
2023-08-09 15:13:11 PDT
Committed
266741@main
(f0bb8782e85b): <
https://commits.webkit.org/266741@main
> Reviewed commits have been landed. Closing PR #16298 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug