Bug 200828 - LocalStorageDatabase should use inline initialization for its data members
Summary: LocalStorageDatabase should use inline initialization for its data members
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-08-16 13:26 PDT by Chris Dumez
Modified: 2019-08-16 17:05 PDT (History)
8 users (show)

See Also:


Attachments
Patch (2.53 KB, patch)
2019-08-16 13:30 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2019-08-16 13:26:52 PDT
LocalStorageDatabase should use inline initialization for its data members.
Comment 1 Chris Dumez 2019-08-16 13:30:10 PDT
Created attachment 376530 [details]
Patch
Comment 2 John Wilander 2019-08-16 16:58:29 PDT
Comment on attachment 376530 [details]
Patch

r=me
Comment 3 EWS 2019-08-16 16:58:37 PDT
Comment on attachment 376530 [details]
Patch

Rejecting attachment 376530 [details] from review queue.

wilander@apple.com does not have reviewer permissions according to https://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/contributors.json.

- If you do not have reviewer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have reviewer rights please correct the error in Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your reviewer rights.
Comment 4 Chris Dumez 2019-08-16 16:59:13 PDT
(In reply to ews-feeder from comment #3)
> Comment on attachment 376530 [details]
> Patch
> 
> Rejecting attachment 376530 [details] from review queue.
> 
> wilander@apple.com does not have reviewer permissions according to
> https://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/
> contributors.json.
> 
> - If you do not have reviewer rights please read
> http://webkit.org/coding/contributing.html for instructions on how to use
> bugzilla flags.
> 
> - If you have reviewer rights please correct the error in
> Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to
> the file (no review needed).  The commit-queue restarts itself every 2
> hours.  After restart the commit-queue will correctly respect your reviewer
> rights.

Darn it :P
Comment 5 John Wilander 2019-08-16 17:00:37 PDT
(In reply to Chris Dumez from comment #4)
> (In reply to ews-feeder from comment #3)
> > Comment on attachment 376530 [details]
> > Patch
> > 
> > Rejecting attachment 376530 [details] from review queue.
> > 
> > wilander@apple.com does not have reviewer permissions according to
> > https://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/
> > contributors.json.
> > 
> > - If you do not have reviewer rights please read
> > http://webkit.org/coding/contributing.html for instructions on how to use
> > bugzilla flags.
> > 
> > - If you have reviewer rights please correct the error in
> > Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to
> > the file (no review needed).  The commit-queue restarts itself every 2
> > hours.  After restart the commit-queue will correctly respect your reviewer
> > rights.
> 
> Darn it :P

Two hours. What year is this, 1995? :P
Comment 6 Chris Dumez 2019-08-16 17:01:41 PDT
(In reply to John Wilander from comment #5)
> (In reply to Chris Dumez from comment #4)
> > (In reply to ews-feeder from comment #3)
> > > Comment on attachment 376530 [details]
> > > Patch
> > > 
> > > Rejecting attachment 376530 [details] from review queue.
> > > 
> > > wilander@apple.com does not have reviewer permissions according to
> > > https://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/
> > > contributors.json.
> > > 
> > > - If you do not have reviewer rights please read
> > > http://webkit.org/coding/contributing.html for instructions on how to use
> > > bugzilla flags.
> > > 
> > > - If you have reviewer rights please correct the error in
> > > Tools/Scripts/webkitpy/common/config/contributors.json by adding yourself to
> > > the file (no review needed).  The commit-queue restarts itself every 2
> > > hours.  After restart the commit-queue will correctly respect your reviewer
> > > rights.
> > 
> > Darn it :P
> 
> Two hours. What year is this, 1995? :P

Yes, let's move to GitHub :D
Comment 7 Chris Dumez 2019-08-16 17:04:09 PDT
Committed r248804: <https://trac.webkit.org/changeset/248804>
Comment 8 Chris Dumez 2019-08-16 17:04:35 PDT
Comment on attachment 376530 [details]
Patch

Landed manually.
Comment 9 Radar WebKit Bug Importer 2019-08-16 17:05:21 PDT
<rdar://problem/54413913>