Bug 205833 - Add a move constructor to IDBResultData
Summary: Add a move constructor to IDBResultData
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sihui Liu
URL:
Keywords: InRadar
: 205816 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-01-06 14:25 PST by Sihui Liu
Modified: 2020-01-08 16:07 PST (History)
12 users (show)

See Also:


Attachments
Patch (2.80 KB, patch)
2020-01-06 14:29 PST, Sihui Liu
no flags Details | Formatted Diff | Diff
Patch (1.37 KB, patch)
2020-01-06 15:34 PST, Sihui Liu
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sihui Liu 2020-01-06 14:25:00 PST
<rdar://problem/58146233>
Comment 1 Sihui Liu 2020-01-06 14:29:41 PST
Created attachment 386887 [details]
Patch
Comment 2 youenn fablet 2020-01-06 15:02:04 PST
Comment on attachment 386887 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=386887&action=review

> Source/WebCore/Modules/indexeddb/shared/IDBResultData.h:91
> +    WEBCORE_EXPORT IDBResultData(IDBResultData&&);

Can we just do IDBResultData(IDBResultData&&) = default given we do this for operator=(&&)?
Or simply remove IDBResultData(const IDBResultData&)?
Comment 3 Sihui Liu 2020-01-06 15:34:50 PST
Created attachment 386899 [details]
Patch
Comment 4 Sihui Liu 2020-01-06 15:35:40 PST
(In reply to youenn fablet from comment #2)
> Comment on attachment 386887 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=386887&action=review
> 
> > Source/WebCore/Modules/indexeddb/shared/IDBResultData.h:91
> > +    WEBCORE_EXPORT IDBResultData(IDBResultData&&);
> 
> Can we just do IDBResultData(IDBResultData&&) = default given we do this for
> operator=(&&)?

Yes.

> Or simply remove IDBResultData(const IDBResultData&)?

Compile error.
Comment 5 WebKit Commit Bot 2020-01-07 14:06:56 PST
The commit-queue encountered the following flaky tests while processing attachment 386899 [details]:

imported/w3c/web-platform-tests/IndexedDB/fire-error-event-exception.html bug 201481 (authors: shvaikalesh@gmail.com and youennf@gmail.com)
The commit-queue is continuing to process your patch.
Comment 6 WebKit Commit Bot 2020-01-07 14:07:49 PST
Comment on attachment 386899 [details]
Patch

Clearing flags on attachment: 386899

Committed r254155: <https://trac.webkit.org/changeset/254155>
Comment 7 WebKit Commit Bot 2020-01-07 14:07:51 PST
All reviewed patches have been landed.  Closing bug.
Comment 8 Sihui Liu 2020-01-08 16:07:21 PST
*** Bug 205816 has been marked as a duplicate of this bug. ***