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.
Alex does this work for you? There's quite a lot of places that need to be touched to make this happen.
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.
Created attachment 318098 [details] WIP Patch WinCairo builds checking GTK/WPE. No XCode changes yet.
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.
Created attachment 318187 [details] Patch
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.
Created attachment 318190 [details] Patch
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.
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.
Created attachment 318300 [details] Patch Rebased
Created attachment 318305 [details] Patch Rebased again
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.
Comment on attachment 318305 [details] Patch rs=me
Comment on attachment 318305 [details] Patch Clearing flags on attachment: 318305 Committed r220857: <http://trac.webkit.org/changeset/220857>
All reviewed patches have been landed. Closing bug.
<rdar://problem/33940723>
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?
(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.
(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.
(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?
I would just go ahead and work on forwarding headers. There's no reason for PAL to operate differently than other WebKit subprojects.
(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. ^^^ 👍👍👍
(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.