233677 – Distinguish contiguous SharedBuffer from non-contiguous one and guarantee immutability: part 3

RESOLVED FIXED 233677

Distinguish contiguous SharedBuffer from non-contiguous one and guarantee immutability: part 3

https://bugs.webkit.org/show_bug.cgi?id=233677

Summary Distinguish contiguous SharedBuffer from non-contiguous one and guarantee imm...

Jean-Yves Avenard [:jya]

Reported 2021-11-30 20:20:53 PST

This is part 3 of bug 233030 and As per https://bugs.webkit.org/show_bug.cgi?id=233030#c40 SharedBuffer base class will be renamed FragmentedSharedBuffer ContiguousSharedBuffer will be renamed SharedBuffer

Attachments
WIP (1017.38 KB, patch) 2021-12-02 03:35 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
WIP (1015.53 KB, patch) 2021-12-02 03:41 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (786.01 KB, patch) 2021-12-03 03:52 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
WIP (788.58 KB, patch) 2021-12-03 04:09 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
WIP (788.58 KB, patch) 2021-12-03 07:23 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
WIP (788.61 KB, patch) 2021-12-03 07:25 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (763.51 KB, patch) 2021-12-03 17:16 PST, Jean-Yves Avenard [:jya]	ews-feeder: commit-queue-	Details Formatted Diff Diff
Patch (716.96 KB, patch) 2021-12-03 17:24 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (767.53 KB, patch) 2021-12-03 20:09 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (767.66 KB, patch) 2021-12-04 17:18 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch (715.60 KB, patch) 2021-12-05 22:43 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (764.77 KB, patch) 2021-12-05 22:44 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (756.75 KB, patch) 2021-12-05 23:27 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (765.23 KB, patch) 2021-12-06 04:37 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch (717.51 KB, patch) 2021-12-06 04:39 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch (716.44 KB, patch) 2021-12-07 01:23 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (767.90 KB, patch) 2021-12-07 01:25 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (760.89 KB, patch) 2021-12-07 04:12 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (766.38 KB, patch) 2021-12-09 05:36 PST, Jean-Yves Avenard [:jya]	ews-feeder: commit-queue-	Details Formatted Diff Diff
Patch for EWS (766.98 KB, patch) 2021-12-09 06:24 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for review (715.05 KB, patch) 2021-12-09 06:26 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for EWS (809.21 KB, patch) 2021-12-13 08:34 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch for review (728.40 KB, patch) 2021-12-13 08:36 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch (729.65 KB, patch) 2021-12-13 15:52 PST, Jean-Yves Avenard [:jya]	ews-feeder: commit-queue-	Details Formatted Diff Diff
Patch (729.65 KB, patch) 2021-12-13 16:30 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Patch (731.86 KB, patch) 2021-12-13 21:45 PST, Jean-Yves Avenard [:jya]	no flags	Details Formatted Diff Diff
Show Obsolete (25) View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2021-12-02 03:20:57 PST

<rdar://problem/85963900>

Jean-Yves Avenard [:jya]

Comment 2 2021-12-02 03:35:32 PST

Created attachment 445700 [details] WIP

Jean-Yves Avenard [:jya]

Comment 3 2021-12-02 03:41:18 PST

Created attachment 445701 [details] WIP

EWS Watchlist

Comment 4 2021-12-02 03:42:24 PST

Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API

Jean-Yves Avenard [:jya]

Comment 5 2021-12-03 03:52:14 PST

Created attachment 445834 [details] Patch for EWS

Jean-Yves Avenard [:jya]

Comment 6 2021-12-03 04:09:53 PST

Created attachment 445838 [details] WIP

Jean-Yves Avenard [:jya]

Comment 7 2021-12-03 07:23:32 PST

Created attachment 445849 [details] WIP

Jean-Yves Avenard [:jya]

Comment 8 2021-12-03 07:25:44 PST

Created attachment 445850 [details] WIP

Jean-Yves Avenard [:jya]

Comment 9 2021-12-03 17:16:34 PST

Created attachment 445929 [details] Patch for EWS

Jean-Yves Avenard [:jya]

Comment 10 2021-12-03 17:24:55 PST

Created attachment 445931 [details] Patch

Jean-Yves Avenard [:jya]

Comment 11 2021-12-03 20:09:13 PST

Created attachment 445947 [details] Patch for EWS

Jean-Yves Avenard [:jya]

Comment 12 2021-12-04 17:18:57 PST

Created attachment 445971 [details] Patch for EWS

Jean-Yves Avenard [:jya]

Comment 13 2021-12-05 22:43:19 PST

Created attachment 445998 [details] Patch Fix test failures

Jean-Yves Avenard [:jya]

Comment 14 2021-12-05 22:44:43 PST

Created attachment 445999 [details] Patch for EWS

Jean-Yves Avenard [:jya]

Comment 15 2021-12-05 23:27:18 PST

Created attachment 446006 [details] Patch for EWS

Jean-Yves Avenard [:jya]

Comment 16 2021-12-05 23:27:55 PST

Comment on attachment 446006 [details] Patch for EWS this patch is for another bug

Jean-Yves Avenard [:jya]

Comment 17 2021-12-06 04:37:29 PST

Created attachment 446026 [details] Patch for EWS rebase

Jean-Yves Avenard [:jya]

Comment 18 2021-12-06 04:39:07 PST

Created attachment 446027 [details] Patch

Jean-Yves Avenard [:jya]

Comment 19 2021-12-07 01:23:51 PST

Created attachment 446135 [details] Patch rebase

Jean-Yves Avenard [:jya]

Comment 20 2021-12-07 01:25:15 PST

Created attachment 446136 [details] Patch for EWS rebase following changes in bug 233030

Jean-Yves Avenard [:jya]

Comment 21 2021-12-07 04:12:40 PST

Created attachment 446153 [details] Patch for EWS

Jean-Yves Avenard [:jya]

Comment 22 2021-12-07 04:15:08 PST

Comment on attachment 446153 [details] Patch for EWS webkit-patch is getting confused by those patch for EWS

Jean-Yves Avenard [:jya]

Comment 23 2021-12-09 05:36:22 PST

Created attachment 446528 [details] Patch for EWS rebase

Jean-Yves Avenard [:jya]

Comment 24 2021-12-09 06:24:36 PST

Created attachment 446535 [details] Patch for EWS rebase 2

Jean-Yves Avenard [:jya]

Comment 25 2021-12-09 06:26:34 PST

Created attachment 446536 [details] Patch for review

Jean-Yves Avenard [:jya]

Comment 26 2021-12-13 08:34:53 PST

Created attachment 447006 [details] Patch for EWS rebase

Jean-Yves Avenard [:jya]

Comment 27 2021-12-13 08:36:34 PST

Created attachment 447007 [details] Patch for review

youenn fablet

Comment 28 2021-12-13 08:53:35 PST

r=me once bots are happy

Jean-Yves Avenard [:jya]

Comment 29 2021-12-13 15:52:20 PST

Created attachment 447077 [details] Patch fix bad earlier rebase

Jean-Yves Avenard [:jya]

Comment 30 2021-12-13 16:30:04 PST

Created attachment 447083 [details] Patch ios build fix

Jean-Yves Avenard [:jya]

Comment 31 2021-12-13 21:45:58 PST

Created attachment 447107 [details] Patch fix rebase error, side fix to GStreamerEMEUtilities and clients, checking for null

youenn fablet

Comment 32 2021-12-14 00:36:18 PST

Comment on attachment 447107 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=447107&action=review > Tools/TestWebKitAPI/Tests/WebCore/SharedBuffer.cpp:44 > +TEST_F(FragmentedSharedBufferTest, createWithContentsOfMissingFile) Are some of these tests worth being duplicated for SharedBuffer as well?

Jean-Yves Avenard [:jya]

Comment 33 2021-12-14 02:14:36 PST

(In reply to youenn fablet from comment #32) > Comment on attachment 447107 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=447107&action=review > > > Tools/TestWebKitAPI/Tests/WebCore/SharedBuffer.cpp:44 > > +TEST_F(FragmentedSharedBufferTest, createWithContentsOfMissingFile) > > Are some of these tests worth being duplicated for SharedBuffer as well? Thanks for the review. I don’t see much values in having extra tests specifically for contiguous buffer no.

EWS

Comment 34 2021-12-14 02:49:16 PST

Committed r287021 (245226@main): <https://commits.webkit.org/245226@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 447107 [details].

Note You need to log in before you can comment on or make changes to this bug.

Status RESOLVED

Resolution FIXED

Priority P2

Severity Normal

Classification Unclassified

Version Other

Hardware Unspecified

OS Unspecified

Product WebKit

Component WebCore Misc.

Assignee

Jean-Yves Avenard [:jya]

Reported

2021-11-30 20:20 PST

Modified

2021-12-14 02:49 PST History

CC List

41 users Show

URL

Keywords InRadar

Depends on

233030 233442

Blocks

233865

Dependencies

tree graph

alecflett
beidson
benjamin
berto
calvaris
cdumez
cgarcia
changseok
eric.carlson
esprehn+autocc
ews-watchlist
galpeter
glenn
gustavo
gyuyoung.kim
hi
hta
japhet
jbedard
jer.noble
joepeck
jsbell
kangil.han
keith_miller
macpherson
mark.lam
menard
mifenton
mmaxfield
msaboff
pangle
philipj
pnormand
rniwa
saam
sergio
tommyw
tzagallo
vjaquez
webkit-bug-importer
youennf