Move platform/sql into PAL. Currently the contents have dependencies in Module/webdatabase and page/MemoryRelease
(In reply to Don Olmstead from comment #0) > Move platform/sql into PAL. > > Currently the contents have dependencies in Module/webdatabase and > page/MemoryRelease SQLiteDatabase.cpp includes MemoryRelease.h, but doesn't do anything with it. So that's easy. I'm not terribly surprised if, at some point, platform/sql got a layering violation towards webdatabase, but it's not immediately obvious what it is just by looking at the code. Perhaps you could actually share details about your discoveries...?
(In reply to Brady Eidson from comment #1) > (In reply to Don Olmstead from comment #0) > > Move platform/sql into PAL. > > > > Currently the contents have dependencies in Module/webdatabase and > > page/MemoryRelease > > SQLiteDatabase.cpp includes MemoryRelease.h, but doesn't do anything with > it. So that's easy. > > I'm not terribly surprised if, at some point, platform/sql got a layering > violation towards webdatabase, but it's not immediately obvious what it is > just by looking at the code. > > Perhaps you could actually share details about your discoveries...? Looks like its just DatabaseAuthorizer that needs to move.
Created attachment 317147 [details] Patch Hopefully fixing xcode
Grr wrong bug. Ignore this