Bug 174192 – [PAL] Move SessionID into PAL

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Don Olmstead]

WebCore::SessionID is required in multiple places within WebCore/platform but is currently at WebCore/page/SessionID.h. For example platform/network/NetworkStorageSession and WebCoreCrossThreadCopier.

It seems like it should reside within pal/network.

[Don Olmstead]

Alex does this work for you? There's quite a lot of places that need to be touched to make this happen.

[Alex Christensen]

Hmmm.  PAL will probably need to use it, and it will probably need to go into PAL, but a SessionID corresponds with a WebsiteDataStore and all forms of persistent storage.  Cookies and HTTP cache are two parts of that, but because it isn't a concept that has only to do with the network, I wouldn't put it in the network directory.

[Don Olmstead]

Created attachment 318098 [details]
WIP Patch

WinCairo builds checking GTK/WPE. No XCode changes yet.

[Build Bot]

Attachment 318098 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:67:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:71:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:73:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:76:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:79:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:80:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:87:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:136:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Storage/StorageProcessProxy.cpp:86:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Storage/StorageProcessProxy.cpp:96:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:110:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:137:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:143:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:149:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:155:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:161:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:167:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:199:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:245:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:264:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:65:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:65:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:70:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:70:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/NetworkProcess/NetworkProcess.cpp:328:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:125:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:140:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebCore/loader/cache/MemoryCache.cpp:275:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm:203:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm:203:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp:183:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp:183:  Missing space inside { }.  [whitespace/braces] [5]
Total errors found: 35 in 194 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Don Olmstead]

Created attachment 318187 [details]
Patch

[Build Bot]

Attachment 318187 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:67:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:71:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:73:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:76:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:79:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:80:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:87:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:136:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Storage/StorageProcessProxy.cpp:86:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Storage/StorageProcessProxy.cpp:96:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:110:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:137:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:143:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:149:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:155:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:161:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:167:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:199:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:245:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:264:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:65:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:65:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:70:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:70:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/NetworkProcess/NetworkProcess.cpp:328:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:125:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:140:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebCore/loader/cache/MemoryCache.cpp:275:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm:203:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm:203:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp:183:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp:183:  Missing space inside { }.  [whitespace/braces] [5]
Total errors found: 35 in 199 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Don Olmstead]

Created attachment 318190 [details]
Patch

[Build Bot]

Attachment 318190 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:67:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:71:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:73:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:76:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:79:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:80:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:87:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:136:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Storage/StorageProcessProxy.cpp:86:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Storage/StorageProcessProxy.cpp:96:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:110:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:137:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:143:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:149:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:155:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:161:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:167:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:199:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:245:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:264:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:65:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:65:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:70:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:70:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/NetworkProcess/NetworkProcess.cpp:328:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:125:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:140:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebCore/loader/cache/MemoryCache.cpp:275:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm:203:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm:203:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp:183:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp:183:  Missing space inside { }.  [whitespace/braces] [5]
Total errors found: 35 in 199 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Don Olmstead]

Comment on attachment 318190 [details]
Patch

This moves SessionID into PAL completely fixing layering violations present in WebCore/platform. Mostly this was just a find/replace operation. Only notable thing is teaching the derived sources script in WebKit about the SessionID's location.

Believe the style errors are just from the style checker not understanding the syntax usage.

[Don Olmstead]

Created attachment 318300 [details]
Patch

Rebased

[Don Olmstead]

Created attachment 318305 [details]
Patch

Rebased again

[Build Bot]

Attachment 318305 [details] did not pass style-queue:


ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:67:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:68:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:71:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:73:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:74:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:76:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:77:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:79:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:80:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:87:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.h:136:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Storage/StorageProcessProxy.cpp:86:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Storage/StorageProcessProxy.cpp:96:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:110:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:137:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:143:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:149:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:155:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:161:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:167:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:199:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:245:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/WebCookieManagerProxy.cpp:264:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:65:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:65:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:70:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/C/WKCookieManager.cpp:70:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/NetworkProcess/NetworkProcess.cpp:328:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:125:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp:140:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebCore/loader/cache/MemoryCache.cpp:275:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm:203:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/Cocoa/WKProcessPool.mm:203:  Missing space inside { }.  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp:183:  Missing space before {  [whitespace/braces] [5]
ERROR: Source/WebKit/UIProcess/API/glib/WebKitCookieManager.cpp:183:  Missing space inside { }.  [whitespace/braces] [5]
Total errors found: 35 in 198 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Antti Koivisto]

Comment on attachment 318305 [details]
Patch

rs=me

[WebKit Commit Bot]

Comment on attachment 318305 [details]
Patch

Clearing flags on attachment: 318305

Committed r220857: <http://trac.webkit.org/changeset/220857>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/33940723>

[Brady Eidson]

Why did SessionID.h move to <pal/identifier/.>?

Why not just <pal/.>?

I am strongly against segregating the paths within the pal exports. We don't do that in WebCore or WTF (with the exception of WTF/text/ which is a special case where there's a particular reason to do so)

Could you either fix this in a followup or roll it out until you have time to revisit?

[Don Olmstead]

(In reply to Brady Eidson from comment #17)
> Why did SessionID.h move to <pal/identifier/.>?
> 
> Why not just <pal/.>?
> 
> I am strongly against segregating the paths within the pal exports. We don't
> do that in WebCore or WTF (with the exception of WTF/text/ which is a
> special case where there's a particular reason to do so)
> 
> Could you either fix this in a followup or roll it out until you have time
> to revisit?

Is the ask to just move the file into the root? or to do something like WebCore where the includes look like <WebCore/*.h>? for WebKit(Legacy)?

I'm fine with doing a followup with whatever the consensus is. I have been trying to organize things when moving them to PAL as a lot of WebCore/platform is a bit of a dumping ground.

[Brady Eidson]

(In reply to Don Olmstead from comment #18)
> (In reply to Brady Eidson from comment #17)
> > Why did SessionID.h move to <pal/identifier/.>?
> > 
> > Why not just <pal/.>?
> > 
> > I am strongly against segregating the paths within the pal exports. We don't
> > do that in WebCore or WTF (with the exception of WTF/text/ which is a
> > special case where there's a particular reason to do so)
> > 
> > Could you either fix this in a followup or roll it out until you have time
> > to revisit?
> 
> Is the ask to just move the file into the root? or to do something like
> WebCore where the includes look like <WebCore/*.h>? for WebKit(Legacy)?
> 

Move to root.

I see now that we're segregating by path in all the <pal> headers. 
<pal/spi>, <pal/system>, <pal/crypto>, etc.

This is silly. If this was a decision made by consensus anywhere, I missed the conversation. I would've participated as a strong "nay"

> I have been trying to organize things when moving them to PAL as a lot of
> WebCore/platform is a bit of a dumping ground.

WebCore itself is a vast dumping ground. But all of the headers still export to a root path.

WTF is a dumping ground, but most of the headers still export to a root path.

WebKit is a dumping ground, but the headers export to a root path.

System headers are a dumping ground, but they mostly export to a root path.

It's unclear to me what the identified value is for engineers in the WebKit project to have to learn longer subdirectories for commonly used headers.

e.g. When I used to include SessionID.h in WebKit, I knew it was <WebCore/SessionID.h>
It would take a day or two to adapt to <pal/SessionID.h>
It would take me a *LONG* time to remember <pal/identifier/SessionID.h>, and I see no value in anybody having to do so.

[Don Olmstead]

(In reply to Brady Eidson from comment #19)
> (In reply to Don Olmstead from comment #18)
> > (In reply to Brady Eidson from comment #17)
> > > Why did SessionID.h move to <pal/identifier/.>?
> > > 
> > > Why not just <pal/.>?
> > > 
> > > I am strongly against segregating the paths within the pal exports. We don't
> > > do that in WebCore or WTF (with the exception of WTF/text/ which is a
> > > special case where there's a particular reason to do so)
> > > 
> > > Could you either fix this in a followup or roll it out until you have time
> > > to revisit?
> > 
> > Is the ask to just move the file into the root? or to do something like
> > WebCore where the includes look like <WebCore/*.h>? for WebKit(Legacy)?
> > 
> 
> Move to root.
> 
> I see now that we're segregating by path in all the <pal> headers. 
> <pal/spi>, <pal/system>, <pal/crypto>, etc.
> 
> This is silly. If this was a decision made by consensus anywhere, I missed
> the conversation. I would've participated as a strong "nay"
> 
> > I have been trying to organize things when moving them to PAL as a lot of
> > WebCore/platform is a bit of a dumping ground.
> 
> WebCore itself is a vast dumping ground. But all of the headers still export
> to a root path.
> 
> WTF is a dumping ground, but most of the headers still export to a root path.
> 
> WebKit is a dumping ground, but the headers export to a root path.
> 
> System headers are a dumping ground, but they mostly export to a root path.
> 
> It's unclear to me what the identified value is for engineers in the WebKit
> project to have to learn longer subdirectories for commonly used headers.
> 
> e.g. When I used to include SessionID.h in WebKit, I knew it was
> <WebCore/SessionID.h>
> It would take a day or two to adapt to <pal/SessionID.h>
> It would take me a *LONG* time to remember <pal/identifier/SessionID.h>, and
> I see no value in anybody having to do so.

A patch is at https://bugs.webkit.org/show_bug.cgi?id=175684 for your review that moves SessionID to the root of the pal directory.

I see your issue with how things are currently looking and since PAL is still being fleshed out I'd like to address your feedback earlier rather than later. Is this something you want addressed on the mailing list so others can chime in or should I just put together a bug that will create forwarding headers which behave in the same way as the <WebCore/*.h> includes and start implementing?

[Michael Catanzaro]

I would just go ahead and work on forwarding headers. There's no reason for PAL to operate differently than other WebKit subprojects.

[Brady Eidson]

(In reply to Michael Catanzaro from comment #21)
> I would just go ahead and work on forwarding headers. There's no reason for
> PAL to operate differently than other WebKit subprojects.

^^^

👍👍👍

[Don Olmstead]

(In reply to Brady Eidson from comment #22)
> (In reply to Michael Catanzaro from comment #21)
> > I would just go ahead and work on forwarding headers. There's no reason for
> > PAL to operate differently than other WebKit subprojects.
> 
> ^^^
> 
> 👍👍👍

Will work on this next https://bugs.webkit.org/show_bug.cgi?id=175705 to get things resolved.