Clean up some includes to make the build a bit faster
Created attachment 379986 [details] Patch
Created attachment 379987 [details] Patch
Created attachment 379988 [details] Patch
Comment on attachment 379988 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=379988&action=review r=me! > Source/WebCore/inspector/InspectorInstrumentation.h:-53 > -#include "InspectorController.h" > #include "InspectorInstrumentationCookie.h" > #include "InspectorInstrumentationPublic.h" > -#include "OffscreenCanvas.h" > #include "Page.h" > #include "StorageArea.h" > -#include "WorkerGlobalScope.h" > -#include "WorkerInspectorController.h" > -#include <JavaScriptCore/JSCInlines.h> Which one of these was worth 80%???
Created attachment 379989 [details] Patch
You need all of them! They all somehow or another get to bad JSC things :D
(In reply to Tim Horton from comment #6) > You need all of them! They all somehow or another get to bad JSC things :D Also the removal of ImageBitmap (which gets JSDOMPromise* IIRC) is like half of it.
Created attachment 379992 [details] Patch
Created attachment 379993 [details] Patch
Created attachment 379994 [details] Patch
https://trac.webkit.org/changeset/250595/webkit
<rdar://problem/55902055>
There was a little bit of build breakage (at least internal macOS builds) which I fixed in r250600.
It looks like this patch https://trac.webkit.org/changeset/250595/webkit broke the open source windows builders: https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Build%29/builds/7182 Log: https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Build%29/builds/7182/steps/compile-webkit/logs/stdio
(In reply to Antoine Quint from comment #13) > There was a little bit of build breakage (at least internal macOS builds) > which I fixed in r250600. Thank you! (In reply to Truitt Savell from comment #14) > It looks like this patch https://trac.webkit.org/changeset/250595/webkit > > broke the open source windows builders: > https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Build%29/ > builds/7182 > > Log: > https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Build%29/ > builds/7182/steps/compile-webkit/logs/stdio Will fix shortly.
Hmm, bots do not think that this was much of a win. Will do more testing and see what I missed. I don't understand how we can mismatch with the new approach :D